[zorg] r188702 - [llvmlab] Move config.phase_config into a local scope.

Michael Gottesman mgottesman at apple.com
Mon Aug 19 13:31:41 PDT 2013


Author: mgottesman
Date: Mon Aug 19 15:31:41 2013
New Revision: 188702

URL: http://llvm.org/viewvc/llvm-project?rev=188702&view=rev
Log:
[llvmlab] Move config.phase_config into a local scope.

Since the phased clang builder and the normal clang builders are in the same
file, when the normal clang builder is imported, PhasedBuilderUtils is imported
as well. Normal lab.llvm.org installations do not support config.phase_config so
this was causing errors.

Modified:
    zorg/trunk/zorg/buildbot/PhasedBuilderUtils.py

Modified: zorg/trunk/zorg/buildbot/PhasedBuilderUtils.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/PhasedBuilderUtils.py?rev=188702&r1=188701&r2=188702&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/PhasedBuilderUtils.py (original)
+++ zorg/trunk/zorg/buildbot/PhasedBuilderUtils.py Mon Aug 19 15:31:41 2013
@@ -12,9 +12,7 @@ from datetime import datetime, date, tim
 import zorg
 
 import config
-import config.phase_config
 reload(config)
-reload(config.phase_config)
 
 class NamedTrigger(Trigger):
     """Trigger subclass which allows overriding the trigger name, and also
@@ -260,6 +258,9 @@ def set_config_option(section, option, d
         return default
 
 def PublishGoodBuild():
+    import config.phase_config
+    reload(config.phase_config)
+    
     artifacts_dir = set_config_option('Master Options', 'artifacts_path',
                                       os.path.expanduser('~/artifacts/'))
     f = buildbot.process.factory.BuildFactory()





More information about the llvm-commits mailing list