summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMichael Fenn <fennm@deshawresearch.com>2013-12-07 00:51:57 -0500
committerMichael Fenn <fennm@deshawresearch.com>2013-12-07 00:51:57 -0500
commitcb884f3c9171d732d875642e32c11ad4ec13f7d1 (patch)
tree83d1ff27b40c8dd2005f80ac84b9343d464900f0 /src
parent356f9f943a34c6d059fec8fcea43b377a977f55c (diff)
downloadbcfg2-cb884f3c9171d732d875642e32c11ad4ec13f7d1.tar.gz
bcfg2-cb884f3c9171d732d875642e32c11ad4ec13f7d1.tar.bz2
bcfg2-cb884f3c9171d732d875642e32c11ad4ec13f7d1.zip
Options: Finalize once
If finalize is called early, then some options will not be parsed but instead always take the default value (observed with reporting.transport). Calling finalize once at the end of the processing lets all options take the values they were assigned in the config file.
Diffstat (limited to 'src')
-rw-r--r--src/lib/Bcfg2/Options/Parser.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/Bcfg2/Options/Parser.py b/src/lib/Bcfg2/Options/Parser.py
index 80f966246..307b767d8 100644
--- a/src/lib/Bcfg2/Options/Parser.py
+++ b/src/lib/Bcfg2/Options/Parser.py
@@ -246,8 +246,8 @@ class Parser(argparse.ArgumentParser):
self._set_defaults()
self.parse_known_args(args=self.argv, namespace=self.namespace)
self._parse_config_options()
- self._finalize()
self._parse_config_options()
+ self._finalize()
# phase 4: fix up <repository> macros
repo = getattr(self.namespace, "repository", repository.default)