Patchwork disable __size_t macro on GNU/kFreeBSD

login
register
mail settings
Submitter Gerald Pfeifer
Date Jan. 31, 2012, 10:29 p.m.
Message ID <alpine.LNX.2.00.1201312328360.2227@gerinyyl.fvgr>
Download mbox | patch
Permalink /patch/138872/
State New
Headers show

Comments

Gerald Pfeifer - Jan. 31, 2012, 10:29 p.m.
On Sun, 29 Jan 2012, Robert Millan wrote:
> Please consider this patch to stddef.h. GNU/kFreeBSD has the same
> problem with __size_t as FreeBSD does, since it inherits many kernel
> headers from FreeBSD.

The patch looks obvious to me, and I'll be happy to apply for
Robert if approved.  Any taker?

Gerald


2012-01-29  Robert Millan  <rmh@gnu.org>

	* ginclude/stddef.h [__FreeBSD_kernel__] (__size_t): Do not define.
Mike Stump - Feb. 1, 2012, 1:06 a.m.
On Jan 31, 2012, at 2:29 PM, Gerald Pfeifer wrote:
> On Sun, 29 Jan 2012, Robert Millan wrote:
>> Please consider this patch to stddef.h. GNU/kFreeBSD has the same
>> problem with __size_t as FreeBSD does, since it inherits many kernel
>> headers from FreeBSD.
> 
> The patch looks obvious to me, and I'll be happy to apply for
> Robert if approved.  Any taker?

Looks obvious to me as well.  I'd say let's put it in...
Robert Millan - Feb. 4, 2012, 7:20 p.m.
El 1 de febrer de 2012 1:06, Mike Stump <mikestump@comcast.net> ha escrit:
> On Jan 31, 2012, at 2:29 PM, Gerald Pfeifer wrote:
>> On Sun, 29 Jan 2012, Robert Millan wrote:
>>> Please consider this patch to stddef.h. GNU/kFreeBSD has the same
>>> problem with __size_t as FreeBSD does, since it inherits many kernel
>>> headers from FreeBSD.
>>
>> The patch looks obvious to me, and I'll be happy to apply for
>> Robert if approved.  Any taker?
>
> Looks obvious to me as well.  I'd say let's put it in...

Is this patch approved, then?
Mike Stump - Feb. 6, 2012, 7:31 p.m.
On Feb 4, 2012, at 11:20 AM, Robert Millan wrote:
> El 1 de febrer de 2012 1:06, Mike Stump <mikestump@comcast.net> ha escrit:
>> On Jan 31, 2012, at 2:29 PM, Gerald Pfeifer wrote:
>>> On Sun, 29 Jan 2012, Robert Millan wrote:
>>>> Please consider this patch to stddef.h. GNU/kFreeBSD has the same
>>>> problem with __size_t as FreeBSD does, since it inherits many kernel
>>>> headers from FreeBSD.
>>> 
>>> The patch looks obvious to me, and I'll be happy to apply for
>>> Robert if approved.  Any taker?
>> 
>> Looks obvious to me as well.  I'd say let's put it in...
> 
> Is this patch approved, then?

I've not seen anyone approve it yet.  I'm not a maintainer for that area, so, I cannot.
Robert Millan - Feb. 11, 2012, 7:53 p.m.
El 6 de febrer de 2012 19:31, Mike Stump <mikestump@comcast.net> ha escrit:
>>> Looks obvious to me as well.  I'd say let's put it in...
>>
>> Is this patch approved, then?
>
> I've not seen anyone approve it yet.  I'm not a maintainer for that area, so, I cannot.

Who can approve it?  Looking at the maintainers file it's not clear to
me which area does this belong to.

Patch

Index: ginclude/stddef.h
===================================================================
--- ginclude/stddef.h	(revision 183670)
+++ ginclude/stddef.h	(working copy)
@@ -200,7 +200,7 @@ 
 #define ___int_size_t_h
 #define _GCC_SIZE_T
 #define _SIZET_
-#if defined (__FreeBSD__) && (__FreeBSD__ >= 5)
+#if (defined (__FreeBSD__) && (__FreeBSD__ >= 5)) || defined(__FreeBSD_kernel__)
 /* __size_t is a typedef on FreeBSD 5!, must not trash it. */
 #else
 #define __size_t