[Top][All Lists]
[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.
- [patch] remove config.status race condition, Lars J. Aas, 2001/02/13
- Re: [patch] remove config.status race condition, Pavel Roskin, 2001/02/15
- Re: [patch] remove config.status race condition, Lars J. Aas, 2001/02/16
- Re: [patch] remove config.status race condition,
Akim Demaille <=
- Re: [patch] remove config.status race condition, Pavel Roskin, 2001/02/23
- Re: [patch] remove config.status race condition, Akim Demaille, 2001/02/23
- Re: [patch] remove config.status race condition, Pavel Roskin, 2001/02/23
- Re: [patch] remove config.status race condition, akim, 2001/02/23
- Re: [patch] remove config.status race condition, Pavel Roskin, 2001/02/24
- Re: [patch] remove config.status race condition, akim, 2001/02/24
- Re: [patch] remove config.status race condition, Akim Demaille, 2001/02/25
- RE: [patch] remove config.status race condition, Tim Van Holder, 2001/02/23
- Re: [patch] remove config.status race condition, Akim Demaille, 2001/02/25
- Re: [patch] remove config.status race condition, Akim Demaille, 2001/02/25