[Automake-NG] [PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Stefano Lattarini
On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this has already been done for ages (at least since 2003). I'll just remove support for it in Automake 1.13. See the patch below. OK? Regards, Stefano

Re: [Automake-NG] [PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Paolo Bonzini
On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini stefano.lattar...@gmail.com wrote: On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this has already been done for ages (at least since 2003). I'll just remove

Re: [Automake-NG] [PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Andrew W. Nosenko
On Wed, Aug 22, 2012 at 10:12 PM, Paolo Bonzini bonz...@gnu.org wrote: On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini stefano.lattar...@gmail.com wrote: On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this

Re: [Automake-NG] [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Paolo Bonzini
Il 22/08/2012 23:52, Stefano Lattarini ha scritto: I'd much rather a mandatory noisy warning period before a feature is completely removed. This would require a new category of warnings that are are unconditionally show, regardless of strictness or any -Wnone option. As usual, patches

Re: [Automake-NG] [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Eric Blake
On 08/22/2012 03:52 PM, Stefano Lattarini wrote: OTOH, I believe developers working on older systems should be ready to install more recent developer tools once in a while. You can't truly expect not to update your Automake installation for 3, 4 years! Oh, _I_ fully wish that RHEL 5 would at

[PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Stefano Lattarini
On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this has already been done for ages (at least since 2003). I'll just remove support for it in Automake 1.13. See the patch below. OK? Regards, Stefano

Re: [PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Paolo Bonzini
On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini stefano.lattar...@gmail.com wrote: On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this has already been done for ages (at least since 2003). I'll just remove

Re: [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Diego Elio Pettenò
On 22/08/2012 12:12, Paolo Bonzini wrote: What about first making the warning visible always, not just with -Wobsolete? And add to the message that support will be removed in 1.14. +1 -- Diego Elio Pettenò — Flameeyes flamee...@flameeyes.eu — http://blog.flameeyes.eu/

Re: [PATCH] {master} compile: remove support for $(INCLUDES) (was: Re: Automake vs. Automake-NG)

2012-08-22 Thread Andrew W. Nosenko
On Wed, Aug 22, 2012 at 10:12 PM, Paolo Bonzini bonz...@gnu.org wrote: On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini stefano.lattar...@gmail.com wrote: On 08/21/2012 06:03 PM, Paolo Bonzini wrote: Looking at GNU Smalltalk, I see: * warn for INCLUDES (vs. AM_CPPFLAGS) Turns out this

Re: [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Eric Blake
On 08/22/2012 09:12 AM, Stefano Lattarini wrote: From 54a49542d417850e646fefe7bad56546a2362449 Mon Sep 17 00:00:00 2001 Message-Id: 54a49542d417850e646fefe7bad56546a2362449.1345648257.git.stefano.lattar...@gmail.com From: Stefano Lattarini stefano.lattar...@gmail.com Date: Wed, 22 Aug 2012

Re: [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Stefano Lattarini
On 08/22/2012 10:28 PM, Eric Blake wrote: On 08/22/2012 09:12 AM, Stefano Lattarini wrote: From 54a49542d417850e646fefe7bad56546a2362449 Mon Sep 17 00:00:00 2001 Message-Id: 54a49542d417850e646fefe7bad56546a2362449.1345648257.git.stefano.lattar...@gmail.com From: Stefano Lattarini

Re: [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Paolo Bonzini
Il 22/08/2012 23:52, Stefano Lattarini ha scritto: I'd much rather a mandatory noisy warning period before a feature is completely removed. This would require a new category of warnings that are are unconditionally show, regardless of strictness or any -Wnone option. As usual, patches

Re: [PATCH] {master} compile: remove support for $(INCLUDES)

2012-08-22 Thread Eric Blake
On 08/22/2012 03:52 PM, Stefano Lattarini wrote: OTOH, I believe developers working on older systems should be ready to install more recent developer tools once in a while. You can't truly expect not to update your Automake installation for 3, 4 years! Oh, _I_ fully wish that RHEL 5 would at