[reprotest] 01/01: reprotest: call sys.exit inside main() not check(), it's more idiomatic

[Ximin Luo]

This is an automated email from the git hooks/post-receive script.

infinity0 pushed a commit to branch master
in repository reprotest.

commit 535685346688e83b767c4635cf5dd62bce321203
Author: Ximin Luo <infinity0 at debian.org>
Date:   Thu Sep 22 19:01:55 2016 +0200

    reprotest: call sys.exit inside main() not check(), it's more idiomatic
---
 reprotest/__init__.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/reprotest/__init__.py b/reprotest/__init__.py
index 6f3ff57..1c4a438 100644
--- a/reprotest/__init__.py
+++ b/reprotest/__init__.py
@@ -341,8 +341,8 @@ def check(build_command, artifact_name, virtual_server_args, source_root,
                       env=env.experiment)
         except Exception:
             traceback.print_exc()
-            sys.exit(2)
-        sys.exit(subprocess.call(['diffoscope', temp_dir + '/control_artifact', temp_dir + '/experiment_artifact']))
+            return 2
+        return subprocess.call(['diffoscope', temp_dir + '/control_artifact', temp_dir + '/experiment_artifact'])
 
 
 COMMAND_LINE_OPTIONS = types.MappingProxyType(collections.OrderedDict([
@@ -462,4 +462,4 @@ def main():
         format='%(message)s', level=30-10*verbosity, stream=sys.stdout)
 
     # print(build_command, artifact, virtual_server_args)
-    check(build_command, artifact, virtual_server_args, source_root, variations)
+    sys.exit(check(build_command, artifact, virtual_server_args, source_root, variations))

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/reproducible/reprotest.git