From owner-svn-src-user@FreeBSD.ORG Tue Nov 13 19:48:51 2012 Return-Path: Delivered-To: svn-src-user@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id D670240A; Tue, 13 Nov 2012 19:48:51 +0000 (UTC) (envelope-from crees@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) by mx1.freebsd.org (Postfix) with ESMTP id B8B738FC0C; Tue, 13 Nov 2012 19:48:51 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.5/8.14.5) with ESMTP id qADJmpDZ009103; Tue, 13 Nov 2012 19:48:51 GMT (envelope-from crees@svn.freebsd.org) Received: (from crees@localhost) by svn.freebsd.org (8.14.5/8.14.5/Submit) id qADJmpHg009102; Tue, 13 Nov 2012 19:48:51 GMT (envelope-from crees@svn.freebsd.org) Message-Id: <201211131948.qADJmpHg009102@svn.freebsd.org> From: Chris Rees Date: Tue, 13 Nov 2012 19:48:51 +0000 (UTC) To: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: svn commit: r242994 - user/crees/rclint X-SVN-Group: user MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-user@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the experimental " user" src tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 Nov 2012 19:48:52 -0000 Author: crees (ports committer) Date: Tue Nov 13 19:48:51 2012 New Revision: 242994 URL: http://svnweb.freebsd.org/changeset/base/242994 Log: Don't choke if some variables are quoted. Example file: ports/devel/git/files/git_daemon.in Add hint about -k option. Revert r242937 (was apparently a WIP) Modified: user/crees/rclint/rclint.py Modified: user/crees/rclint/rclint.py ============================================================================== --- user/crees/rclint/rclint.py Tue Nov 13 19:45:13 2012 (r242993) +++ user/crees/rclint/rclint.py Tue Nov 13 19:48:51 2012 (r242994) @@ -66,9 +66,9 @@ class Db: err = self.error(key) if err: if level == 'error': - logging.error('[%s:%d]: %s ' % ("error", num+1, err)) + logging.error('[%d]: %s ' % (num+1, err)) if level == 'warn': - logging.warn('[%s:%d]: %s ' % ("warning", num+1, err)) + logging.warn('[%d]: %s ' % (num+1, err)) if verbosity > 0: print((textwrap.fill(self.explanation(key), initial_indent='==> ', @@ -78,7 +78,7 @@ class Db: self.count += 1 if self.count > 10 and beaucoup_errors == False: hint = ' Try rerunning with -v option for extra details.' if verbosity == 0 else '' - logging.error('Error threshold reached-- further errors are unlikely to be helpful. Fix the errors and rerun.' + hint) + logging.error('Error threshold reached-- further errors are unlikely to be helpful. Fix the errors and rerun. The -k option will cause rclint to continue for as many errors as it finds.' + hint) exit() def warn(self, key, num=-1, level='warn'): @@ -114,6 +114,12 @@ class Statement: return False return True + def get_value(self): + if self.quoted(): + return self.value[1:-1] + else: + return self.value + class Variable(Statement): def __init__(self, lines, number): line = lines[number] @@ -396,7 +402,7 @@ def do_rclint(filename): logging.debug('Checking for rcvar set correctly') for var in lineobj['Variable']: if var.name == 'name': - progname = var.value + progname = var.get_value() elif var.name == 'rcvar': try: if progname + '_enable' not in var.value: @@ -422,7 +428,7 @@ def do_rclint(filename): logging.debug('Checking $name agrees with PROVIDE and filename') fn = filename[:-3] if filename[-3:] == '.in' else filename fn = fn.split('/')[-1].replace('-', '_') - n = get(lineobj['Variable'], 'name').value + n = get(lineobj['Variable'], 'name').get_value() rcordervars = [] for r in lineobj['Rcorder']: if r.type != 'PROVIDE':