Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Rich Felker
On Tue, Oct 27, 2015 at 12:16:16AM +, Joseph Myers wrote: > On Mon, 26 Oct 2015, Rich Felker wrote: > > > On Mon, Oct 26, 2015 at 11:42:37PM +, Joseph Myers wrote: > > > On Mon, 26 Oct 2015, Rich Felker wrote: > > > > > > > musl explicitly does not support using a mix of libc headers and

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Joseph Myers
On Mon, 26 Oct 2015, Rich Felker wrote: > On Mon, Oct 26, 2015 at 11:42:37PM +, Joseph Myers wrote: > > On Mon, 26 Oct 2015, Rich Felker wrote: > > > > > musl explicitly does not support using a mix of libc headers and > > > compiler-provided freestanding headers. While there may be > > > >

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Rich Felker
On Mon, Oct 26, 2015 at 11:42:37PM +, Joseph Myers wrote: > On Mon, 26 Oct 2015, Rich Felker wrote: > > > musl explicitly does not support using a mix of libc headers and > > compiler-provided freestanding headers. While there may be > > In that case the GCC ports for musl should define USER_

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Joseph Myers
On Mon, 26 Oct 2015, Rich Felker wrote: > musl explicitly does not support using a mix of libc headers and > compiler-provided freestanding headers. While there may be In that case the GCC ports for musl should define USER_H = $(EXTRA_HEADERS) like t-openbsd does. (Of course that won't work for

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Joseph Myers
On Mon, 26 Oct 2015, Szabolcs Nagy wrote: > > FLT_ROUNDS is an ordinary compiler bug (bug 30569), should be fixable > > reasonably straightforwardly as outlined at > > , probably within say a > > week's work if most architecture-specific changes ar

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Doug Evans
On Fri, Oct 23, 2015 at 11:34 AM, Szabolcs Nagy wrote: > On 23/10/15 18:39, Doug Evans wrote: >> >> On Fri, Oct 23, 2015 at 10:08 AM, Bernd Schmidt >> wrote: >>> >>> >>> On 10/21/2015 09:00 PM, Doug Evans wrote: I happened to notice local prefixes not working with musl. Fixes

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Rich Felker
On Mon, Oct 26, 2015 at 12:32:01PM +, Szabolcs Nagy wrote: > On 23/10/15 21:20, Joseph Myers wrote: > >On Fri, 23 Oct 2015, Szabolcs Nagy wrote: > > > >>i think bsd libcs do the same, compiler headers interfering > >>with libc headers is problematic (e.g. FLT_ROUNDS is wrong > >>in gcc float.h,

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-26 Thread Szabolcs Nagy
On 23/10/15 21:20, Joseph Myers wrote: On Fri, 23 Oct 2015, Szabolcs Nagy wrote: i think bsd libcs do the same, compiler headers interfering with libc headers is problematic (e.g. FLT_ROUNDS is wrong in gcc float.h, applications shouldn't see that), i'm not sure FLT_ROUNDS is an ordinary comp

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-23 Thread Joseph Myers
On Fri, 23 Oct 2015, Szabolcs Nagy wrote: > i think bsd libcs do the same, compiler headers interfering > with libc headers is problematic (e.g. FLT_ROUNDS is wrong > in gcc float.h, applications shouldn't see that), i'm not sure FLT_ROUNDS is an ordinary compiler bug (bug 30569), should be fixab

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-23 Thread Szabolcs Nagy
On 23/10/15 18:39, Doug Evans wrote: On Fri, Oct 23, 2015 at 10:08 AM, Bernd Schmidt wrote: On 10/21/2015 09:00 PM, Doug Evans wrote: I happened to notice local prefixes not working with musl. Fixes thusly. Index: config/linux.h ===

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-23 Thread Bernd Schmidt
On 10/23/2015 07:36 PM, Doug Evans wrote: ​The only significant different AFAICT is that GCC_INCLUDE_DIR is moved to later (last). Why this is is briefly described in the intro comment: config/linux.h: ​ /* musl avoids problematic includes by rearranging the include directories. * Unfortunat

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-23 Thread Doug Evans
On Fri, Oct 23, 2015 at 10:08 AM, Bernd Schmidt wrote: > > On 10/21/2015 09:00 PM, Doug Evans wrote: >> >> I happened to notice local prefixes not working with musl. >> Fixes thusly. > > >> Index: config/linux.h >> === >> --- config/l

Re: [PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-23 Thread Bernd Schmidt
On 10/21/2015 09:00 PM, Doug Evans wrote: I happened to notice local prefixes not working with musl. Fixes thusly. Index: config/linux.h === --- config/linux.h(revision 229130) +++ config/linux.h(working copy) @@ -174,6 +1

[PATCH] Add missing INCLUDE_DEFAULTS_MUSL_LOCAL

2015-10-21 Thread Doug Evans
Heya. I happened to notice local prefixes not working with musl. Fixes thusly. 2015-10-21 Doug Evans * config/linux.h (INCLUDE_DEFAULTS): Add INCLUDE_DEFAULTS_MUSL_LOCAL. Index: config/linux.h === --- config/linux.h