[PATCH v2 1/2] Split up endian.h to minimize exposure of BYTE_ORDER.

Joseph Myers joseph@codesourcery.com
Wed Oct 2 16:30:00 GMT 2019


This appears to have broken the testsuite for all platforms.  How was it 
tested?

FAIL: conform/POSIX2008/arpa/inet.h/conform
FAIL: conform/POSIX2008/netdb.h/conform
FAIL: conform/POSIX2008/netinet/in.h/conform
FAIL: conform/UNIX98/arpa/inet.h/conform
FAIL: conform/UNIX98/netdb.h/conform
FAIL: conform/UNIX98/netinet/in.h/conform
FAIL: conform/XOPEN2K/arpa/inet.h/conform
FAIL: conform/XOPEN2K/netdb.h/conform
FAIL: conform/XOPEN2K/netinet/in.h/conform
FAIL: conform/XOPEN2K8/arpa/inet.h/conform
FAIL: conform/XOPEN2K8/netdb.h/conform
FAIL: conform/XOPEN2K8/netinet/in.h/conform
FAIL: conform/XPG42/arpa/inet.h/conform
FAIL: conform/XPG42/netdb.h/conform
FAIL: conform/XPG42/netinet/in.h/conform

By keeping <endian.h> in networking headers, while removing __USE_MISC 
conditionals from that header, it introduces namespace violations to the 
networking headers in all standards modes.  You'll need to restore the 
__USE_MISC conditionals in <endian.h>.

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Libc-alpha mailing list