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

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,

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 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 > Date: Wed, 22 Aug 2012 16:40:15 +0200 > Subje

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 wrote: > On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini > 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 do

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 Paolo Bonzini
On Wed, Aug 22, 2012 at 5:12 PM, Stefano Lattarini 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 support for i

[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 --