[libcxx-commits] [PATCH] D68295: [libcxx] [test] Use host system separators when setting PATH in config.py

Sergej Jaskiewicz via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Oct 1 13:57:44 PDT 2019


broadwaylamb created this revision.
Herald added a reviewer: EricWF.
Herald added subscribers: libcxx-commits, ldionne, christof.
Herald added a project: libc++.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D68295

Files:
  libcxx/utils/libcxx/test/config.py


Index: libcxx/utils/libcxx/test/config.py
===================================================================
--- libcxx/utils/libcxx/test/config.py
+++ libcxx/utils/libcxx/test/config.py
@@ -1205,6 +1205,6 @@
         if 'PATH' not in dest_env:
             dest_env['PATH'] = new_path
         else:
-            split_char = ';' if self.target_info.is_windows() else ':'
+            split_char = ';' if platform.system() == 'Windows' else ':'
             dest_env['PATH'] = '%s%s%s' % (new_path, split_char,
                                            dest_env['PATH'])


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68295.222697.patch
Type: text/x-patch
Size: 575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20191001/eb7b8b27/attachment-0001.bin>


More information about the libcxx-commits mailing list