This is the mail archive of the libc-hacker@sourceware.org mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] check_pf.c: Include alloca.h

From: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
To: drepper at redhat dot com
Cc: libc-hacker at sources dot redhat dot com
Date: Sat, 07 Jul 2007 09:09:24 +0900 (JST)
Subject: Re: [PATCH] check_pf.c: Include alloca.h
References: <20070707.065436.35682509.kkojima@rr.iij4u.or.jp><468EC0D1.5050104@redhat.com><20070707.074224.38692079.kkojima@rr.iij4u.or.jp>

> Ok.  I'd like to see why alloca.h isn't included indirectly
> in that case on SH.

It turned out that it's because nptl/sysdeps/sh/tls.h missed
including some standard headers.  Uli, could you please approve
the appended nptl patch?  Sorry for not noticing this ealier.

Regards,
	kaz
--
2007-07-06  Kaz Kojima  <kkojima@rr.iij4u.or.jp>

	* sysdeps/sh/tls.h: Include stdlib.h, list.h, sysdep.h and
	kernel-features.h.

--- ORIG/libc/nptl/sysdeps/sh/tls.h	2007-06-09 15:11:40.000000000 +0900
+++ LOCAL/libc/nptl/sysdeps/sh/tls.h	2007-07-07 08:03:19.000000000 +0900
@@ -26,6 +26,10 @@
 # include <stdbool.h>
 # include <stddef.h>
 # include <stdint.h>
+# include <stdlib.h>
+# include <list.h>
+# include <sysdep.h>
+# include <kernel-features.h>
 
 /* Type for the dtv.  */
 typedef union dtv

Follow-Ups:
- Re: [PATCH] check_pf.c: Include alloca.h
  - From: Ulrich Drepper

References:
- [PATCH] check_pf.c: Include alloca.h
  - From: Kaz Kojima
- Re: [PATCH] check_pf.c: Include alloca.h
  - From: Ulrich Drepper
- Re: [PATCH] check_pf.c: Include alloca.h
  - From: Kaz Kojima

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]