Re: [PATCH 0/2] Re: ifdef expessions in Makefile.am

2009-12-22 Thread Paolo Bonzini
On 12/22/2009 09:38 AM, Joakim Tjernlund wrote: bonz...@gnu.org wrote on 22/12/2009 09:16:59: From: Paolo Bonzini Here is where I was at. After that it was not immediate how to use a tag-dependent cache variable. Strictly speaking however using a cache variable is not needed to make the PIC

Re: [PATCH 0/2] Re: ifdef expessions in Makefile.am

2009-12-22 Thread Joakim Tjernlund
bonz...@gnu.org wrote on 22/12/2009 09:16:59: > > From: Paolo Bonzini > > Here is where I was at. After that it was not immediate how to > use a tag-dependent cache variable. Strictly speaking however > using a cache variable is not needed to make the PIC test > overridable. > > The patch is qui

[PATCH 0/2] Re: ifdef expessions in Makefile.am

2009-12-22 Thread bonzini
From: Paolo Bonzini Here is where I was at. After that it was not immediate how to use a tag-dependent cache variable. Strictly speaking however using a cache variable is not needed to make the PIC test overridable. The patch is quite risky though. Paolo Paolo Bonzini (2): split -Wl test

Re: ifdef expessions in Makefile.am

2009-12-21 Thread Joakim Tjernlund
Ralf Wildenhues wrote on 21/12/2009 22:14:51: > > * Joakim Tjernlund wrote on Sun, Dec 20, 2009 at 03:58:47PM CET: > > Yes, thanks for the script layout. What would be even > > cooler is if one could add a custom macro in acinclude.m4 > > that would override the libtool.m4 macro creating the -fPIC

Re: ifdef expessions in Makefile.am

2009-12-21 Thread Ralf Wildenhues
* Joakim Tjernlund wrote on Sun, Dec 20, 2009 at 03:58:47PM CET: > Yes, thanks for the script layout. What would be even > cooler is if one could add a custom macro in acinclude.m4 > that would override the libtool.m4 macro creating the -fPIC option > and replace it with -fpic instead. No, it woul

Re: ifdef expessions in Makefile.am

2009-12-20 Thread Joakim Tjernlund
Ralf Wildenhues wrote on 18/12/2009 08:45:41: > From: Ralf Wildenhues > To: Joakim Tjernlund > Cc: Peter Johansson , automake@gnu.org > Date: 18/12/2009 08:45 > Subject: Re: ifdef expessions in Makefile.am > > * Joakim Tjernlund wrote on Thu, Dec 17, 2009 at 11:29:19PM CE

Re: ifdef expessions in Makefile.am

2009-12-18 Thread Ralf Wildenhues
* Joakim Tjernlund wrote on Thu, Dec 17, 2009 at 11:29:19PM CET: > if @TEST@ then > SUBDIRS+= dir1/dir2/@TEST@ > SUBDIRS+= dir3/dir2/@TEST@ > SUBDIRS+= dir4/@TEST@ > > endif The syntax is if TEST SUBDIRS += ... endif and there should not be any indentation of `if' and `endif', and

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Joakim Tjernlund
Peter Johansson wrote on 18/12/2009 00:43:06: > From: Peter Johansson > To: Joakim Tjernlund > Cc: automake@gnu.org > Date: 18/12/2009 00:42 > Subject: Re: ifdef expessions in Makefile.am > > Joakim Tjernlund wrote: > > I tried the example you pointed me to,

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Joakim Tjernlund
Joakim Tjernlund/Transmode wrote on 18/12/2009 00:32:11: > From: Joakim Tjernlund/Transmode > To: Peter Johansson > Cc: automake@gnu.org > Date: 18/12/2009 00:32 > Subject: Re: ifdef expessions in Makefile.am > > Peter Johansson wrote on 18/12/2009 00:05:46: > > >

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Peter Johansson
Joakim Tjernlund wrote: I tried the example you pointed me to, added this to configure.ac: AM_CONDITIONAL([COND_OPT], [test "$want_opt" = yes]) and got: # > autoconf configure.ac:32: error: possibly undefined macro: AM_CONDITIONAL If this token and others are legitimate, please use m4_pat

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Joakim Tjernlund
Peter Johansson wrote on 18/12/2009 00:05:46: > From: Peter Johansson > To: Joakim Tjernlund > Cc: automake@gnu.org > Date: 18/12/2009 00:05 > Subject: Re: ifdef expessions in Makefile.am > > Hi Joakim, > > Joakim Tjernlund wrote: > > Thanks,

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Peter Johansson
Hi Joakim, Joakim Tjernlund wrote: Thanks, but I can't make that work for me AM_CONDITIONAL seems to be an automake 1.11 feature That seems inaccurate because I've used AM_CONDITIONAL since 2004. and the AC_SUBST variant does not quite fit Basically I want: if @TEST@ then SUBDIRS+= dir1/

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Harlan Stenn
Dave Hart wrote: > You're running up against something else. AM_CONDITIONAL goes back > some time, and has worked splendidly for the NTP reference > implementation built using Automake 1.10. and even automake 1.9, and probably earlier versions as well. H

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Dave Hart
On Thu, Dec 17, 2009 at 22:29 UTC, Joakim Tjernlund wrote: > AM_CONDITIONAL seems to be an automake 1.11 feature You're running up against something else. AM_CONDITIONAL goes back some time, and has worked splendidly for the NTP reference implementation built using Automake 1.10. Cheers, Dave Ha

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Joakim Tjernlund
Peter Johansson wrote on 17/12/2009 22:52:35: > From: Peter Johansson > To: Joakim Tjernlund > Cc: automake@gnu.org > Date: 17/12/2009 22:52 > Subject: Re: ifdef expessions in Makefile.am > > Joakim Tjernlund wrote: > > I want to do something that is similar to: >

Re: ifdef expessions in Makefile.am

2009-12-17 Thread Peter Johansson
Joakim Tjernlund wrote: I want to do something that is similar to: SUBDIRS+= dir1/dir2/@TEST@ but only if @TEST@ is defined. Any suggestions? http://www.gnu.org/software/automake/manual/automake.html#Conditional-Subdirectories -- Peter Johansson svndigest maintainer, http://dev.thep.lu.

ifdef expessions in Makefile.am

2009-12-17 Thread Joakim Tjernlund
I want to do something that is similar to: SUBDIRS+= dir1/dir2/@TEST@ but only if @TEST@ is defined. Any suggestions? I am stuck with automake 1.10.2 Jocke