Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Thomas De Schampheleire
Hi Dmitry, On Wed, Dec 7, 2016 at 4:18 PM, Dmitry V. Levin wrote: > On Wed, Dec 07, 2016 at 02:58:35PM +0300, Dmitry V. Levin wrote: >> On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: >> > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: >>

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Dmitry V. Levin
On Wed, Dec 07, 2016 at 02:58:35PM +0300, Dmitry V. Levin wrote: > On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: > > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: > [...] > > > The correct fix is going to be more complex, e.g. > > > - move all

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Dmitry V. Levin
Hi Thomas, On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: [...] > > The correct fix is going to be more complex, e.g. > > - move all gl_WARN_ADD/WARN_CFLAGS related code from configure.ac to a > > separate m4

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-06 Thread Thomas De Schampheleire
Hi Dmitry, On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: > Hi, > > On Tue, Oct 18, 2016 at 01:42:44PM +0200, Thomas De Schampheleire wrote: >> From: Thomas De Schampheleire >> >> Commit dc427d50d96b15c9a64d7e78d97ce4d194622a63

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-11-16 Thread Dmitry V. Levin
Hi, On Tue, Oct 18, 2016 at 01:42:44PM +0200, Thomas De Schampheleire wrote: > From: Thomas De Schampheleire > > Commit dc427d50d96b15c9a64d7e78d97ce4d194622a63 enabled a number of extra > warning flags through configure.ac. The configure script will determine

[PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-10-18 Thread Thomas De Schampheleire
From: Thomas De Schampheleire Commit dc427d50d96b15c9a64d7e78d97ce4d194622a63 enabled a number of extra warning flags through configure.ac. The configure script will determine dynamically if CC supports these flags before adding them to WARN_CFLAGS. ioctlsort