[PATCH] D30231: Replace backslashes with forward slashes

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 15:21:16 PST 2017


zturner created this revision.

CMake doesn't seem to like when paths have backslashes in them, so replacing them all with forward slashes.

Not sure if there's a better way to accomplish this.  This is leading to failures such as here:

http://lab.llvm.org:8011/builders/sanitizer-windows/builds/6620/steps/cmake_1/logs/stdio


https://reviews.llvm.org/D30231

Files:
  zorg/buildbot/builders/SanitizerBuilderWindows.py


Index: zorg/buildbot/builders/SanitizerBuilderWindows.py
===================================================================
--- zorg/buildbot/builders/SanitizerBuilderWindows.py
+++ zorg/buildbot/builders/SanitizerBuilderWindows.py
@@ -142,7 +142,7 @@
                           env=Property('slave_env')))
 
     # Get absolute path to clang-cl.
-    clang_cl = "%(workdir)s/" + build_dir + "/bin/clang-cl"
+    clang_cl = ("%(workdir)s/" + build_dir + "/bin/clang-cl").replace("\\", "/")
     f.addStep(ShellCommand(name='cmake',
                            command=[cmake, "-G", "Ninja", "../llvm",
                                "-DCMAKE_BUILD_TYPE="+config,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30231.89291.patch
Type: text/x-patch
Size: 665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170221/bf75f389/attachment.bin>


More information about the llvm-commits mailing list