[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Using $bindir in configure.ac
From: |
Stepan Kasal |
Subject: |
Re: Using $bindir in configure.ac |
Date: |
Mon, 10 Jan 2005 15:51:24 +0100 |
User-agent: |
Mutt/1.4.1i |
Hello,
> On Sat, Jan 08, 2005 at 11:43:25AM +0100, Magnus Therning wrote:
> > http://www.gnu.org/software/ac-archive/htmldoc/ac_define_dir.html
a few minutes ago, I wrote about a problem (the mail is cited below).
I noticed another one: the code doesn't handle spaces in path names
correctly. So I suggest to correct the assignments too.
An updated patch is attached to this mail.
Have a nice day,
Stepan Kasal
---- my previous mail:
> This code contains the following:
>
> ifelse($3, ,
> AC_DEFINE_UNQUOTED($1, "$ac_define_dir"),
> AC_DEFINE_UNQUOTED($1, "$ac_define_dir", $3))
>
> First, the `$3' argument is underquoted. It is a description, so it could
> easily contain a comma. Second, I see no need for the `ifelse'.
>
> So I propose the patch attached to this mail.
> BTW: what does it mean that the macro is in the legacy section of the CVS?
>
> Regards,
> Stepan
ac-archive-ac_define_dir.patch
Description: Text document
- Re: Using $bindir in configure.ac, Stepan Kasal, 2005/01/10
- Re: Using $bindir in configure.ac,
Stepan Kasal <=
- Re: Using $bindir in configure.ac, Andreas Schwab, 2005/01/10
- Re: Using $bindir in configure.ac, Stepan Kasal, 2005/01/10
- Re: Using $bindir in configure.ac, Andreas Schwab, 2005/01/10
- Re: Using $bindir in configure.ac, Stepan Kasal, 2005/01/11
- Re: Using $bindir in configure.ac, Peter Simons, 2005/01/14
- Re: Using $bindir in configure.ac, Stepan Kasal, 2005/01/17
- Re: Using $bindir in configure.ac, Peter Simons, 2005/01/17