[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

I fixed this once already [Was Re: [PATCH] bootstrap: obey --no-git.]

From: Gary V. Vaughan
Subject: I fixed this once already [Was Re: [PATCH] bootstrap: obey --no-git.]
Date: Mon, 15 Aug 2011 03:38:25 +0700

Hi All,

On 14 Aug 2011, at 05:45, Paul Eggert wrote:
> On 08/13/2011 03:17 PM, Bruno Haible wrote:
>> Jim, Eric, Paul, you are the maintainers of 'bootstrap'. Any objection
>> against James' patch [1]?
> No, it looks OK to me; please apply.  If we run into problems
> we can fix them later.

Is my complete rewrite of bootstrap (which already has what appears
to be an earlier name for the same option, `--skip-git', at Eric's
request) unsuitable?

No disrespect to the incumbent script, which by Paul's own admission
sort of grew organically in a few projects which eventually accreted
in gnulib, but my rewrite was quite carefully designed and written
specifically to be maintainable, extensible and understandable.

And it runs over 30% faster than the current version (last I checked -
if the slurp function has been removed from gnulib's existing
bootstrap, then mine probably lost most of its edge in execution time).

And it successfully uses my M4 based method of extracting various
filenames and directory paths from configure.ac, which obvious works
far more reliably than sed. I'd like to see libtoolize and
autoreconf and others adopt this method so that we can use macro
expansions and whatever quoting we need without worrying about
the current tools choking in sed expressions.

And it is very well commented, now with a good first draft texinfo
chapter for @including into the gnulib documentation.

No organic accretion here.

I'm attaching the latest version, along with the texinfo documentation
in hope of consideration for adoption into gnulib so that people other
than me can benefit.

Please tell me to stop bugging the list if you're tired of my
mentioning this from time to time. On the other hand, if you just
need more proof that this one is working as well as I say, I'll add
it on a topic branch to the gnulib using projects I have commit
rights to, and update the bootstrap.confs I wrote for them last
year provided someone will take a look afterwards, with the intention
of agreeing to supercede the existing bootstrap if everything works
as advertised.

I feel a bit like the elephant in the room...

Gary V. Vaughan (gary AT gnu DOT org)

Attachment: bootstrap.tar.bz2
Description: BZip2 compressed data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]