Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Erik de Castro Lopo
Hi all, I am a big fan of 'make distcheck' and also a fan of compiling with gcc's -Werrror flag. Most of my configure.ac scripts also have a --enable-gcc-werror options so that I can use -Werror while developing but ship tarballs without it. So now I had this great idea. Wouldn't it be great if I

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Ralf Wildenhues
Hello Ericm * Erik de Castro Lopo wrote on Sat, Feb 09, 2008 at 09:47:06AM CET: > So now I had this great idea. Wouldn't it be great if I could tell > 'make distcheck' use './configure --enable-gcc-werror'! Use DISTCHECK_CONFIGURE_FLAGS. Cheers, Ralf

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Braden McDaniel
On Sat, 2008-02-09 at 19:47 +1100, Erik de Castro Lopo wrote: > Hi all, > > I am a big fan of 'make distcheck' and also a fan of compiling with > gcc's -Werrror flag. Most of my configure.ac scripts also have a > --enable-gcc-werror options so that I can use -Werror while developing > but ship tar

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Erik de Castro Lopo
Braden McDaniel wrote: > > Anyone have a clue on how to do this? > > DISTCHECK_CONFIGURE_FLAGS = --enable-gcc-werror > > in your top-level Makefile.am. Wow, that is a very beautiful thing. Thanks Braden! Cheers, Erik -- - Erik d

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Bob Friesenhahn
On Sat, 9 Feb 2008, Erik de Castro Lopo wrote: So now I had this great idea. Wouldn't it be great if I could tell 'make distcheck' use './configure --enable-gcc-werror'! Anyone have a clue on how to do this? While this can certainly be done (as explained in subsequent email), you might want

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Bob Friesenhahn
On Sat, 9 Feb 2008, Ralf Wildenhues wrote: OTOH, Erik could also put this in his Makefile.am: my-distcheck: $(MAKE) $(AM_MAKEFLAGS) DISTCHECK_CONFIGURE_FLAGS=--enable-gcc-werror That seems much better from an overall package portability standpoint. Which is one reason I'm hesitatin

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Sat, Feb 09, 2008 at 05:11:29PM CET: > On Sat, 9 Feb 2008, Erik de Castro Lopo wrote: >> >> So now I had this great idea. Wouldn't it be great if I could tell >> 'make distcheck' use './configure --enable-gcc-werror'! >> >> Anyone have a clue on how to do this? >

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sat, Feb 09, 2008 at 05:58:01PM CET: > On Sat, 9 Feb 2008, Ralf Wildenhues wrote: > >> Which is one reason I'm hesitating to enable the option dist-lzma for >> the Automake package itself. > > I assumed that if dist-lzma was requested that if the lzma tools were > not a

Re: ACLOCAL_AMFLAGS problems

2008-02-09 Thread Ralf Wildenhues
Hello Ruben, Apologies for the delay, I must have missed this somehow: * Ruben Henner Zilibowitz wrote on Tue, Dec 18, 2007 at 01:59:44PM CET: > I have run into a problem where the following line: > ACLOCAL_AMFLAGS = -I m4 > in t

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Bob Friesenhahn
On Sat, 9 Feb 2008, Ralf Wildenhues wrote: If the Automake option `dist-lzma' is enabled, the `dist' rule (and thus also the `distcheck' rule) will require successful lzma tarball creation. The rationale here is of course: if the package maintainer is actually doing the distribution, then heck,

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sat, Feb 09, 2008 at 07:34:31PM CET: > > Thanks for the clarification. To be 100% correct, each of the created > dist files should be verified to see that its content passes the > distcheck tests. If *that* were still a concern for a compression tool (as opposed to

Re: Modifying CFLAGS for 'make distcheck'

2008-02-09 Thread Bob Friesenhahn
On Sat, 9 Feb 2008, Ralf Wildenhues wrote: If *that* were still a concern for a compression tool (as opposed to various vendor `tar' programs), then heck it should not be promoted at all for wider use. No, I don't think each package using Automake should be turned into a regression test suite f