[zorg] r305623 - Assert that -DCMAKE_INSTALL_PREFIX is not specified for multi-stage builders.

Galina Kistanova via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 18:21:35 PDT 2017


Author: gkistanova
Date: Fri Jun 16 20:21:35 2017
New Revision: 305623

URL: http://llvm.org/viewvc/llvm-project?rev=305623&view=rev
Log:
Assert that -DCMAKE_INSTALL_PREFIX is not specified for multi-stage builders.

Modified:
    zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py

Modified: zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py?rev=305623&r1=305622&r2=305623&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py (original)
+++ zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py Fri Jun 16 20:21:35 2017
@@ -263,7 +263,7 @@ def getClangWithLTOBuildFactory(
     # in the extra_configure_args. We set them internally as needed.
     # TODO: assert extra_configure_args.
     install_prefix_specified = any(a.startswith('-DCMAKE_INSTALL_PREFIX=') for a in extra_configure_args)
-    assert True, "Please do not explicitly specify the install prefix for multi-stage build."
+    assert not install_prefix_specified, "Please do not explicitly specify the install prefix for multi-stage build."
 
     # Prepare environmental variables. Set here all env we want everywhere.
     merged_env = {




More information about the llvm-commits mailing list