about summary refs log tree commit diff
path: root/conform/Makefile
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2016-04-26 09:55:47 +0000
committerJoseph Myers <joseph@codesourcery.com>2016-04-26 09:55:47 +0000
commitbf074726151d032bc345eaaa56e0858b1677e2ee (patch)
tree87eadb467ca1f420c8d72139d33327467fca4761 /conform/Makefile
parent12404bb04e1e86052fb22005e6c05ab3d1ced46e (diff)
downloadglibc-bf074726151d032bc345eaaa56e0858b1677e2ee.tar.gz
glibc-bf074726151d032bc345eaaa56e0858b1677e2ee.tar.xz
glibc-bf074726151d032bc345eaaa56e0858b1677e2ee.zip
Define off_t in stdio.h for XOPEN2K.
The header conformance test for stdio.h for XOPEN2K fails because the
header does not define the off_t type, used in the expected
declarations for fseeko and ftello.

The absence of this type is not actually strictly a bug (hence no bug
report being filed in Bugzilla), since POSIX didn't require the type
to be declared in this header until the 2008 edition.  However, the
glibc convention in such cases - where the type falls under the
general *_t POSIX reservation, and so it's OK to define it for all
POSIX versions - is to make the headers self-contained in this regard
even for the older POSIX versions not requiring the type to be defined
despite including other declarations depending on the type.  Thus,
this patch adjusts the condition in the header and removes the XFAIL
(rather than adapting the expectation to work when the functions are
declared using __off_t without off_t being defined).

Tested for x86_64 and x86 (testsuite, and that installed stripped
shared libraries are unchanged by the patch).

	* libio/stdio.h (off_t): Define if [__USE_XOPEN2K], not
	[__USE_XOPEN2K8].
	[__USE_LARGEFILE64] (off64_t): Likewise.
	* conform/Makefile (test-xfail-XOPEN2K/stdio.h/conform): Remove
	variable.
Diffstat (limited to 'conform/Makefile')
-rw-r--r--conform/Makefile1
1 files changed, 0 insertions, 1 deletions
diff --git a/conform/Makefile b/conform/Makefile
index e3baec7c9f..7af4b2fee8 100644
--- a/conform/Makefile
+++ b/conform/Makefile
@@ -216,7 +216,6 @@ test-xfail-XOPEN2K/mqueue.h/conform = yes
 test-xfail-XOPEN2K/semaphore.h/conform = yes
 test-xfail-XOPEN2K/signal.h/conform = yes
 test-xfail-XOPEN2K/stdarg.h/conform = yes
-test-xfail-XOPEN2K/stdio.h/conform = yes
 test-xfail-XOPEN2K/sys/wait.h/conform = yes
 test-xfail-XOPEN2K/ucontext.h/conform = yes
 test-xfail-POSIX2008/arpa/inet.h/conform = yes