Merge PR #131: Move setting branch info from cmdline option so it is …

…persisted
smarr · Jan 5, 2020 · af7490c · af7490c
2 parents c1544cf + e795979
commit af7490c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/rebench/persistence.py b/rebench/persistence.py
@@ -73,6 +73,9 @@ def get(self, filename, configurator):
             if configurator.use_rebench_db and 'repo_url' in configurator.rebench_db:
                 _source['repoURL'] = configurator.rebench_db['repo_url']
 
+            if configurator.options and configurator.options.branch:
+                _source['branchOrTag'] = configurator.options.branch
+
             p = _FilePersistence(filename, self, configurator.discard_old_data, self._ui)
             self._ui.debug_output_info('ReBenchDB enabled: {e}\n', e=configurator.use_rebench_db)
 
@@ -91,9 +94,6 @@ def get(self, filename, configurator):
                         "\n\n"
                         "Use the --experiment option to set the name.")
 
-                if configurator.options.branch:
-                    _source['branchOrTag'] = configurator.options.branch
-
                 db = _ReBenchDB(configurator.rebench_db['db_url'],
                                 configurator.rebench_db['project_name'],
                                 configurator.options.experiment_name,