-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[libc++] Mark _XOPEN_SOURCE test as unsupported on FreeBSD #128950
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The test otherwise fails on FreeBSD, which wasn't noticed when originally landing the patch that added the test because FreeBSD CI was disabled at that moment.
@llvm/pr-subscribers-libcxx Author: Louis Dionne (ldionne) ChangesThe test otherwise fails on FreeBSD, which wasn't noticed when originally landing the patch that added the test because FreeBSD CI was disabled at that moment. Full diff: https://github.com/llvm/llvm-project/pull/128950.diff 1 Files Affected:
diff --git a/libcxx/test/libcxx/xopen_source.gen.py b/libcxx/test/libcxx/xopen_source.gen.py
index 3f2686483730a..d4a3651181ca7 100644
--- a/libcxx/test/libcxx/xopen_source.gen.py
+++ b/libcxx/test/libcxx/xopen_source.gen.py
@@ -43,6 +43,9 @@
// recent value of _XOPEN_SOURCE.
// UNSUPPORTED: LIBCXX-AIX-FIXME
+// This test fails on FreeBSD for an unknown reason.
+// UNSUPPORTED: LIBCXX-FREEBSD-FIXME
+
{lit_header_restrictions.get(header, '')}
{lit_header_undeprecations.get(header, '')}
|
Errors we see without this:
and
@emaste Any clue what's going on? |
I think I have seen errors like the second one when I have made my attempt at implementing |
Merging the patch since it did fix the specific failure I wanted to fix on FreeBSD and it makes us closer to a green CI. |
The test otherwise fails on FreeBSD, which wasn't noticed when originally landing the patch that added the test because FreeBSD CI was disabled at that moment.
The test otherwise fails on FreeBSD, which wasn't noticed when originally landing the patch that added the test because FreeBSD CI was disabled at that moment.