[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Problem with AC_CONFIG_FILES targets that contain variables
From: |
Sander Niemeijer |
Subject: |
Re: Problem with AC_CONFIG_FILES targets that contain variables |
Date: |
Fri, 22 Aug 2008 13:12:42 +0200 |
Hi Ralf,
On 22 aug 2008, at 10:42, Ralf Wildenhues wrote:
Currently the part that may be misleading is the part where it says
that
one is encouraged to avoid using 'AC_CONFIG_FOOS([$my_foos])'. This
phrasing of the sentence suggests that it may still be valid to do
so.
In fact, it is not. Running configure works. But, just as my
example, a
config.status on either fooo or foooo does not. Even adding an
initialisation of my_foos to init-cmds wont help. I would have
guessed
that maybe running "config.status 'fooo foooo'" would then still have
worked, but even that doesn't seem to (config.status apparently
splits
up all command line arguments, even if they were quoted, so it
gives an
'error: invalid argument: fooo'.).
Erm, we fixed a regression that occurred in 2.62 in this area
recently.
Can you be more specific and show the test case that fails?
Just follow the example from Section 4.6:
---
AC_INIT(foo, 1.0)
my_foos="fooo foooo"
AC_CONFIG_FILES([$my_foos])
AC_OUTPUT
---
or with your 'fix'
---
AC_INIT(foo, 1.0)
my_foos="fooo foooo"
AC_CONFIG_FILES([$my_foos], [], [my_foos='$my_foos'])
AC_OUTPUT
---
After running configure, try running either:
$ ./config.status fooo
or
$ ./config.status 'fooo foooo'
Both will fail with:
---
config.status: error: invalid argument: fooo
---
Which Autoconf version did you use?
2.62, as mentioned in my first post.
I would rephrase the section to explicitly forbid users to use the
my_foos construct as is currently mentioned and generally advise
users
not to use variables in the tag section. It would be good to mention
what users can expect if they would still try to do it (i.e.
configure
works, but config.status wont).
Only if we can't get it fixed.
This was assuming you would follow what you said: 'I don't think we
will fix this'
Best regards,
Sander