Akim Demaille wrote:
> To me this is featurism. I really believe one single address for all the
> configure related questions is way enough. If people have suggestion to
> make, them the real installed software seems to me like the proper place to
> report such things.
>
> I'd like to stick to a single email address.
I don't have really strong feelings about this, but I'd like to hear
other opinions however. My main concern is that, as always, if you don't give
people the means to do what they want to do (to a certain extent, obvisouly,
and *this* is not going very far in that direction anyway), they'll end up
hacking their script with explicit calls to Autoconf's internal routines. We
all know that this is shooting ourselves in the foot with respect to backward
compatibility maintenance.
> In addition, the string we use is quite standardized.
Sounds like a Gnits concern more than a GNU one to me ?
> | configure --version:
> | ,----
> | | Configure script for Toto 1.0, generated by Autoconf 2.14a.
> | |
>
> This is not right, the proper formatting should be what it is
> currently:
>
> % ./configure --version
> configure (Toto 1.0) 2.14a
>
> See the GNU coding standards.
Oups. I didn't know the GCS specified this, sorry. This looks horrible
to me, that's why I proposed to change it.
> To answer your question before it comes :), yes, we follow the GNU coding
> standards in Autoconf, and there is no debate about this.
Nah, I wasn't going to ask :-)
>
> | @@ -2004,7 +2012,8 @@
> | [AC_DIVERT_ONCE([HELP_ENABLE], [[
> | Optional Features:
> | --disable-FEATURE do not include FEATURE (same as --enable-FEATURE=no)
> | - --enable-FEATURE[=ARG] include FEATURE [ARG=yes]]])dnl
> | + --enable-FEATURE[=ARG] include FEATURE [ARG=yes]
> | +]])dnl
> | AC_DIVERT_ONCE([HELP_ENABLE], [$2])dnl
> | # Check whether --enable-$1 or --disable-$1 was given.
> | if test "[${enable_]patsubst([$1], -, _)+set}" = set; then
>
> Why is that?
Err, that's because there's a similar blank line for before `--with-*'
option listing.
> I guess you didn't update before running your diff.
Yes I did, but maybe in the middle of one of your bunches... wait...
ah, I see:
| (AC_ARG_WITH): Remove spurious newline.
Damn! I liked this blank lines :-)
--
/ / _ _ Didier Verna http://www.inf.enst.fr/~verna/
- / / - / / /_/ / EPITA / LRDE mailto:[EMAIL PROTECTED]
/_/ / /_/ / /__ / 14-16 rue Voltaire Tel. +33 (1) 44 08 01 77
94276 Kremlin-Bicêtre cedex Fax. +33 (1) 44 08 01 99
