[libcxx-commits] [libcxx] f11ad99 - [libcxx][lit] Fixing libcxx test failures on Windows (#194752)

[via libcxx-commits]

Author: Haowei
Date: 2026-04-29T09:41:12-06:00
New Revision: f11ad99f08fc64a93ba9b6f8d2c7faa8ecbdcd52

URL: https://github.com/llvm/llvm-project/commit/f11ad99f08fc64a93ba9b6f8d2c7faa8ecbdcd52
DIFF: https://github.com/llvm/llvm-project/commit/f11ad99f08fc64a93ba9b6f8d2c7faa8ecbdcd52.diff

LOG: [libcxx][lit] Fixing libcxx test failures on Windows (#194752)

PR#194368 changed how line breaks are handles on Windows and it broke
several libcxx tests on Windows, including
libcxx/test/std/localization/locale.categories/facet.numpunct/
locale.numpunct.byname/thousands_sep.pass.cpp
This patch addresses this issue.

Added: 
    

Modified: 
    libcxx/test/selftest/dsl/dsl.sh.py
    libcxx/utils/libcxx/test/features/localization.py

Removed: 
    


################################################################################
diff  --git a/libcxx/test/selftest/dsl/dsl.sh.py b/libcxx/test/selftest/dsl/dsl.sh.py
index c35a3bb6a48a4..95b97fde2dadf 100644
--- a/libcxx/test/selftest/dsl/dsl.sh.py
+++ b/libcxx/test/selftest/dsl/dsl.sh.py
@@ -160,7 +160,9 @@ def test_valid_program_returns_output_newline_handling(self):
         #include <cstdio>
         int main(int, char**) { std::printf("FOOBAR\\n"); return 0; }
         """
-        self.assertEqual(dsl.programOutput(self.config, source), "FOOBAR\n")
+        self.assertEqual(
+            dsl.programOutput(self.config, source), "FOOBAR%s" % os.linesep
+        )
 
     def test_valid_program_returns_no_output(self):
         source = """
@@ -224,14 +226,14 @@ def test_caching_is_not_too_aggressive(self):
             compileFlags + " -DMACRO=1",
         )
         output1 = dsl.programOutput(self.config, source)
-        self.assertEqual(output1, "MACRO=1\n")
+        self.assertEqual(output1, "MACRO=1%s" % os.linesep)
 
         self.config.substitutions[compileFlagsIndex] = (
             "%{compile_flags}",
             compileFlags + " -DMACRO=2",
         )
         output2 = dsl.programOutput(self.config, source)
-        self.assertEqual(output2, "MACRO=2\n")
+        self.assertEqual(output2, "MACRO=2%s" % os.linesep)
 
     def test_program_stderr_is_not_conflated_with_stdout(self):
         # Run a program that produces stdout output and stderr output too, making

diff  --git a/libcxx/utils/libcxx/test/features/localization.py b/libcxx/utils/libcxx/test/features/localization.py
index 88e240798d271..95e1bb5c1afa4 100644
--- a/libcxx/utils/libcxx/test/features/localization.py
+++ b/libcxx/utils/libcxx/test/features/localization.py
@@ -144,5 +144,5 @@ def _getLocaleFlagsAction(cfg, locale, alts, members):
             f"%{{LOCALE_CONV_{valid_define_name}_{member.upper()}}}",
             lambda cfg, value=value: f"'L\"{value}\"'",
         )
-        for member, value in zip(members, localeconv_info.split("\n"))
+        for member, value in zip(members, localeconv_info.splitlines())
     ]