[PATCH] update configure.in and use autoreconf

Michael Biebl mbiebl at gmail.com
Tue May 1 23:00:26 PDT 2007


2007/5/2, David Zeuthen <david at fubar.dk>:
> On Fri, 2007-04-27 at 01:32 +0300, Guillem Jover wrote:
> > Update the autotools files to not use obsolete macros and variables
> > and add proper quotation. Switch to autoreconf which will call any
> > needed program. Also autoreconf -Wall should be clean now (except
> > for external problems).
>
> Great, thanks a lot, committed. How does this relate to Doug's other
> patch for configure.in?

Please consider, that running autoreconf --force --install will
overwrite the COPYING file.

We also should raise the automake dependency to  1.8 or 1.9
configure.in: AM_INIT_AUTOMAKE([gnu 1.9])
It doesn't build with automake 1.5.

I also noticed that the current logic in doc/man/Makefile.am is wrong:
The *.in files should be added to EXTRA_DIST in any case, not
depending on MAN_PAGES_ENABLED or HAVE_POLKIT.

We also shouldn't distribute man_MANS in EXTRA_DIST, because these
files are autobuilt.
It is inconsistent to remove them on clean and at the same time adding
them to EXTRA_DIST.

Michael


-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?


More information about the hal mailing list