Some autoconf cleanups

Ondrej Zajicek santiago at
Tue Mar 7 00:17:41 CET 2017

On Sat, Mar 04, 2017 at 06:13:16PM +0100, Ruben Kerkhof wrote:
> The goal of this series is to make it possible to run autoreconf on a git checkout.
> It does this by moving to and fixes deprecation warnings
> and errors generated by autoheader.


Thanks for the cleanup patches, our configure script is old and not much
kept up-to-date. I have some questions w.r.t. your patches:

It seems that (by removing sysdep/ the patchset not only
make it possible to run autoreconf, but make it necessary. I am not
familiar with that tool, could you briefly describe what are advantages
of using it?

>  rename aclocal.m4 => m4/bird.m4 (100%)

Is this necessary? I have an aversion to boilerplate directories
containing just one file.

> ...

Shouldn't quotes be also there? Esp. when other patches added plenty of
quotes to other macro expressions.

> -AC_INIT(conf/confbase.Y)

According to autoconf doc, AC_INIT should have at least arguments
'package, version'? Well, they were missing in the old code too ...

Elen sila lumenn' omentielvo

Ondrej 'Santiago' Zajicek (email: santiago at
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3,
"To err is human -- to blame it on a computer is even more so."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <>

More information about the Bird-users mailing list