autoconf-patches
[Top][All Lists]
Advanced

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

Re: [patch] remove config.status race condition


From: Akim Demaille
Subject: Re: [patch] remove config.status race condition
Date: 23 Feb 2001 20:05:26 +0100
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Cuyahoga Valley)

"Lars J. Aas" <address@hidden> writes:

> Hey, I sent is a patch that fixes a *bug*!  Why do I have to justify
> myself to you on this one?

Agreed.

Nonetheless I don't agree with patch :)

First of all, it is not obvious at all that config.status should run
this, since Tim still did not convince me that AS_LN_S was a good
thing.  As is, AS_LN_S is just *useless*.  We were happier before.

Since AS_LN_S is not solving the problems config.status has, there is
no reason to have it run by config.status, and in fact, not even by
configure.  I think including this guy was an error, or, at best, the
current implementation is just not enough.

That's one means to get rid of the problem: rm -rf.

Second means consists in just using $$.  We have room for this:
s/conftest/conf$$/, I really can't understand why we'd need something
more complex.

Given that the second choice is the easiest and laziest approach,
unless someone objects, that's what I'll do.



reply via email to

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