[PATCH] fix datarootdir warnings by autoconf 2.61

[sean finney]

On Wed, Jun 17, 2009 at 10:50:41AM +0200, Andreas Ericsson wrote:
> Ethan Galstad wrote:
> >> Andreas Ericsson schrieb:
> > 
> > The only thing anyone should be mucking with when making changes is
> > configure.in.  The "configure" script has been in CVS for ages, and
> > there's no reason to take it out now.  I don't really think there's an
> > absolute requirement that all the core devs use the same autoconf
> > version, as long as its somewhat recent.  Makes it a hell of a lot
> > easier to just to a CVS checkout and run ./configure, which helps with
> > automated scripts.
> > 
> 
> There are two problems with this.
> 
> The first is patch review. If someone makes a change to configure.in
> and re-generates configure from it with a more recent version of autoconf
> the patch will be *huge*, but the actual change might be 3-4 lines.

i would say at the least it would be good form to split them into seperate
commits, so that one could see the "actual" change vs the "10000 lines of
noise" change.

> Personally I like what the plugins do about this, shipping a microscopic
> script that just issues the autoconf commands so one doesn't have to
> remember them.

yeah.  i'd go so far as to call this a "best practice" for using autoconf,
i.e. shipping a "bootstrap.sh" or "autogen.sh" which does some sanity checks
and then generates the autofoo stuff.

for 3rd party distributers (i.e. debian and other distros), as soon as you
have to make one change in the build system, the out-of-the-box configure
scripts become pretty burdensome to deal with.

i'd say the existance of a hypothetical autogen script plus nightly snapshots
ought to placate most of the arguments behind having ./configure in the VCS.

granted i'm not the most active with nagios these days, but i have to
deal with this type of problem all the time and it's a major PITA to
deal with so i figured i'd just chip in my 0.16 SEK :)


	sean
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <https://www.monitoring-lists.org/archive/developers/attachments/20090617/81f53f32/attachment.sig>
-------------- next part --------------
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
-------------- next part --------------
_______________________________________________
Nagios-devel mailing list
Nagios-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-devel