Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-10-01 Thread Michael Forney
On 2018-10-01, David Phillips wrote: > On Mon, Oct 01, 2018 at 06:35:13PM -0700, Michael Forney wrote: >> Rather than the if-else, I think we should always include sys/types.h, >> and also sys/sysmacros.h on glibc. I also think the comment is >> unnecessary. Do you mind if I apply with those two c

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-10-01 Thread David Phillips
On Mon, Oct 01, 2018 at 06:35:13PM -0700, Michael Forney wrote: > On 2018-10-01, David Phillips wrote: > > Hi, > > > > Bumping this patch since sbase master now fails to build against > > glibc 2.28 > > > > Let me know if there are any improvements that could be made. > > > > Thanks, > > David >

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-10-01 Thread Michael Forney
On 2018-10-01, David Phillips wrote: > Hi, > > Bumping this patch since sbase master now fails to build against > glibc 2.28 > > Let me know if there are any improvements that could be made. > > Thanks, > David I wish the #ifdef wasn't necessary, but it seems like the only way to use major/minor

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-10-01 Thread David Phillips
On Sun, Jul 08, 2018 at 10:51:44PM +1200, David Phillips wrote: > > Reworked patch encompassing this behaviour is attached. Would > be interested to see builds against glibc pre-2.25. > > Let me know what you think. > > Thanks, > David Hi, Bumping this patch since sbase master now fails to bui

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-08 Thread David Phillips
On Mon, Jul 02, 2018 at 11:58:54PM +1200, David Phillips wrote: > On Mon, Jul 02, 2018 at 01:20:42PM +0200, Quentin Rameau wrote: > > Ok, the makedev(3) manpage from the man-pages states this indeed: > > > > The BSDs expose the definitions for these macros via . > > Depending on the version, glibc

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread David Phillips
On Mon, Jul 02, 2018 at 01:20:42PM +0200, Quentin Rameau wrote: > Ok, the makedev(3) manpage from the man-pages states this indeed: > > The BSDs expose the definitions for these macros via . > Depending on the version, glibc also exposes definitions for these > macros from that header file if suit

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread Quentin Rameau
> > On glibc, major, minor, and makedev are all defined in > > sys/sysmacros.h with types.h only including this for historical > > reasons. A future release of glibc will remove this behaviour, > > meaning that major, minor, and makedev will no longer be defined > > for us without including sysmacr

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread David Phillips
On Mon, Jul 02, 2018 at 10:16:50AM +0200, Quentin Rameau wrote: > > On glibc, major, minor, and makedev are all defined in > > sys/sysmacros.h with types.h only including this for historical > > reasons. A future release of glibc will remove this behaviour, > > meaning that major, minor, and makede

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread David Phillips
On Mon, Jul 02, 2018 at 10:02:09AM +0200, Hiltjo Posthuma wrote: > > Did you test it with musl too and preferably other platforms? Unfortunately not; I don't currently have access to non-glibc Linux installations at the moment, nor BSDs. Perhaps users of those systems might be able to chime in at

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread Quentin Rameau
> On glibc, major, minor, and makedev are all defined in > sys/sysmacros.h with types.h only including this for historical > reasons. A future release of glibc will remove this behaviour, > meaning that major, minor, and makedev will no longer be defined > for us without including sysmacros.h. Cou

Re: [hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread Hiltjo Posthuma
On Mon, Jul 02, 2018 at 07:48:03PM +1200, David Phillips wrote: > On glibc, major, minor, and makedev are all defined in > sys/sysmacros.h with types.h only including this for historical > reasons. A future release of glibc will remove this behaviour, > meaning that major, minor, and makedev will n

[hackers] [PATCH] [sbase] Include sysmacros.h directly rather than types.h

2018-07-02 Thread David Phillips
On glibc, major, minor, and makedev are all defined in sys/sysmacros.h with types.h only including this for historical reasons. A future release of glibc will remove this behaviour, meaning that major, minor, and makedev will no longer be defined for us without including sysmacros.h. --- ls.c | 2