[libcxx-commits] [PATCH] D120546: [libcxx] [test] Fix dsl.sh.py to work on Windows.

Martin Storsjö via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sun Feb 27 12:10:30 PST 2022


mstorsjo updated this revision to Diff 411699.
mstorsjo added a comment.

Rebased, after committing acf20001a01119ce02d5f2b0282e5a7c966ca12c <https://reviews.llvm.org/rGacf20001a01119ce02d5f2b0282e5a7c966ca12c> on its own. (No action needed here right now, awaiting verdict on D120623 <https://reviews.llvm.org/D120623> first.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120546

Files:
  libcxx/test/libcxx/selftest/dsl/dsl.sh.py
  libcxx/utils/libcxx/test/dsl.py


Index: libcxx/utils/libcxx/test/dsl.py
===================================================================
--- libcxx/utils/libcxx/test/dsl.py
+++ libcxx/utils/libcxx/test/dsl.py
@@ -95,6 +95,10 @@
     res = ('', res.output, 127, None)
   (out, err, exitCode, timeoutInfo) = res
 
+  if platform.system() == 'Windows':
+    out = out.replace('\r\n', '\n')
+    err = err.replace('\r\n', '\n')
+
   # TODO: As a temporary workaround until https://reviews.llvm.org/D81892 lands, manually
   #       split any stderr output that is included in stdout. It shouldn't be there, but
   #       the Lit internal shell conflates stderr and stdout.
Index: libcxx/test/libcxx/selftest/dsl/dsl.sh.py
===================================================================
--- libcxx/test/libcxx/selftest/dsl/dsl.sh.py
+++ libcxx/test/libcxx/selftest/dsl/dsl.sh.py
@@ -6,8 +6,6 @@
 #
 #===----------------------------------------------------------------------===##
 
-# XFAIL: LIBCXX-WINDOWS-FIXME
-
 # Note: We prepend arguments with 'x' to avoid thinking there are too few
 #       arguments in case an argument is an empty string.
 # RUN: %{python} %s x%S x%T x%{substitutions}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120546.411699.patch
Type: text/x-patch
Size: 1165 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220227/a9d8abda/attachment.bin>


More information about the libcxx-commits mailing list