[Pm-utils] [PATCH] Much simpler autogen.sh for our needs

Dan Nicholson dbn.lists at gmail.com
Thu Feb 7 11:24:29 PST 2008


On Feb 7, 2008 11:15 AM, Till Maas <opensource at till.name> wrote:
> On Thu February 7 2008, Dan Nicholson wrote:
> > The current autogen.sh script had way too much baggage for what we use
> > in pm-utils. My guess is it was copied from a GNOME package.
> >
> > Anyway, this slims down autogen.sh to just use autoreconf, which will do
> > the right thing. This autogen.sh is a virtual copy of the "official"
> > script used in Xorg.
>
> Imho at least a warning that one needs to install autoconf, when there is no
> autoreconf may be helpful for people who do not know autotools. Does
> autoreconf also provide useful warnings when the other binaries are missing,
> that it needs, e.g. automake, like the old autogen.sh script did?

Yes it does. What was in autogen.sh was just reinventing the wheel of
what autoreconf already does. And since it's written by the autotools
developers (comes with autoconf), it's much more likely to do the
right thing.

$ sudo mv /usr/bin/autoconf{,.off}
Password:
$ autoreconf -iv
autoreconf: Entering directory `.'
autoreconf: configure.in: not using Gettext
autoreconf: running: aclocal
autoreconf: configure.in: tracing
sh: /usr/bin/autoconf: No such file or directory
autoreconf: configure.in: not using Libtool
autoreconf: running: /usr/bin/autoconf
Can't exec "/usr/bin/autoconf": No such file or directory at
/usr/share/autoconf/Autom4te/FileUtils.pm line 290.
autoreconf: failed to run /usr/bin/autoconf: No such file or directory

I suppose we could add an error message that the autoconf package is
needed if autoreconf isn't found.

--
Dan


More information about the Pm-utils mailing list