X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=daklib%2Fchecks.py;h=664bca3e1c966a28d3ea2c7fd3b0708ff8b794ac;hb=1d7f6d35706b847dd00bec1d8694f8676ec215a8;hp=bfc8a17297d1f80b7c757ee74d4523e481d7b300;hpb=e6b1c633b7127f1d2ef938b7ea2181ce9e184906;p=dak.git diff --git a/daklib/checks.py b/daklib/checks.py index bfc8a172..664bca3e 100644 --- a/daklib/checks.py +++ b/daklib/checks.py @@ -147,7 +147,7 @@ class SignatureAndHashesCheck(Check): .format(filename, os.path.basename(e.filename))) raise except InvalidHashException as e: - raise Reject('{0}: {1}'.format(what, unicode(e))) + raise Reject('{0}: {1}'.format(filename, unicode(e))) class ChangesCheck(Check): """Check changes file for syntax errors.""" @@ -638,11 +638,14 @@ class LintianCheck(Check): changespath = os.path.join(upload.directory, changes.filename) try: - if cnf.unprivgroup: - cmd = "sudo -H -u {0} -- /usr/bin/lintian --show-overrides --tags-from-file {1} {2}".format(cnf.unprivgroup, temp_filename, changespath) - else: - cmd = "/usr/bin/lintian --show-overrides --tags-from-file {0} {1}".format(temp_filename, changespath) - result, output = commands.getstatusoutput(cmd) + cmd = [] + + user = cnf.get('Dinstall::UnprivUser') or None + if user is not None: + cmd.extend(['sudo', '-H', '-u', user]) + + cmd.extend(['/usr/bin/lintian', '--show-overrides', '--tags-from-file', temp_filename, changespath]) + result, output = commands.getstatusoutput(" ".join(cmd)) finally: os.unlink(temp_filename)