[PATCH] D83503: [buildbot] Annotated builder tweaks

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 16:53:39 PDT 2020


This revision was automatically updated to reflect the committed changes.
Closed by commit rZORG521b2cea3e93: [buildbot] Annotated builder tweaks (authored by tra).

Repository:
  rZORG LLVM Github Zorg

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83503/new/

https://reviews.llvm.org/D83503

Files:
  zorg/buildbot/builders/AnnotatedBuilder.py


Index: zorg/buildbot/builders/AnnotatedBuilder.py
===================================================================
--- zorg/buildbot/builders/AnnotatedBuilder.py
+++ zorg/buildbot/builders/AnnotatedBuilder.py
@@ -11,7 +11,8 @@
     env=None,
     extra_args=None,
     timeout=1200,
-    is_legacy_mode=False):
+    is_legacy_mode=False,
+    checkout_llvm_sources=True):
     """
     Returns a new build factory that uses AnnotatedCommand, which
     allows the build to be run by version-controlled scripts that do
@@ -76,19 +77,22 @@
         src_dir='llvm-zorg',
         alwaysUseLatest=True)
 
-    f.addGetSourcecodeSteps()
-
+    if checkout_llvm_sources:
+      f.addGetSourcecodeSteps()
 
     extra_args_with_props = [WithProperties(arg) for arg in extra_args]
     # Explicitly use '/' as separator, because it works on *nix and Windows.
-    script_path = "../llvm-zorg/zorg/buildbot/builders/annotated/%s" % (script)
+    if script.startswith('/'):
+      command = [script]
+    else:
+      script_path = "../llvm-zorg/zorg/buildbot/builders/annotated/%s" % (script)
+      command = ["python", script_path, WithProperties("--jobs=%(jobs:-)s")]
+    command += extra_args_with_props
+
     f.addStep(AnnotatedCommand(name="annotate",
                                description="annotate",
                                timeout=timeout,
                                haltOnFailure=True,
-                               command=["python",
-                                        script_path,
-                                        WithProperties("--jobs=%(jobs:-)s")]
-                                        + extra_args_with_props,
+                               command=command,
                                env=merged_env))
     return f


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83503.279969.patch
Type: text/x-patch
Size: 1772 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200722/90290aa7/attachment.bin>


More information about the llvm-commits mailing list