[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnumach: configure: add --with-version-suffix=STRING
From: |
Thomas Schwinge |
Subject: |
Re: [PATCH] gnumach: configure: add --with-version-suffix=STRING |
Date: |
Thu, 27 Sep 2012 09:23:19 +0200 |
User-agent: |
Notmuch/0.9-101-g81dad07 (http://notmuchmail.org) Emacs/23.3.1 (i486-pc-linux-gnu) |
Hi!
On Mon, 24 Sep 2012 02:04:05 +0200, Guillem Jover <guillem@hadrons.org> wrote:
> On Mon, 2012-09-24 at 00:30:39 +0200, Pino Toscano wrote:
> > Alle lunedì 24 settembre 2012, Thomas Schwinge ha scritto:
> > > On Mon, 24 Sep 2012 00:04:20 +0200, Pino Toscano wrote:
> > > > +# An optional suffix for the version string.
> > > > +AC_MSG_CHECKING([for the suffix for the version string])
> > > > +AC_ARG_WITH(version-suffix,
> > > > AC_HELP_STRING([--with-version-suffix=STRING], +
> > > > [append STRING to the version string]),
> > > > + [PACKAGE_VERSION_SUFFIX=$withval])
> > > > +AC_MSG_RESULT([$PACKAGE_VERSION_SUFFIX])
> > > > +AC_SUBST([PACKAGE_VERSION_SUFFIX])
> > >
> > > Does the AC_MSG_CHECKING and AC_MSG_RESULT add any useful information
> > > to the build log? (You should know which option you're configuring
> > > with.)
>
> Maybe use AC_MSG_NOTICE instead?
>
> > IMHO it makes the log sligtly more useful, since that option somehow
> > modifies gnumach so it could be useful to know which options affected
> > its build; also a log could be provided by other people, so the more
> > information about its build it provides, the better.
>
> Yeah, I agree.
I'm not aware of configure scripts typically doing that? I'd suggest to
drop that entirely, but don't insist on that.
> > > Is --with-version-suffix the standard name for this option?
> >
> > No idea, to be honest :)
>
> I don't think there's any such standard name for this.
Right, probably not too wide-spread. GCC has:
--with-pkgversion=PKG Use PKG in the version string in place of "GCC"
Without setting that:
i686-pc-gnu-gcc (GCC) 4.7.0 20110905 (experimental)
With setting it:
gcc (Debian 4.7.1-7) 4.7.1
(Search for case-insensitive pkgversion in config/acx.m4,
gcc/Makefile.in, gcc/toplev.c, gcc/version.c.) Would that seem
reasonable for GNU Mach, too? Otherwise, go with your variant.
Grüße,
Thomas
pgpddQjbm4zJy.pgp
Description: PGP signature