Re: [patch] remove config.status race condition

2001-02-17 Thread Lars J. Aas
On Fri, Feb 16, 2001 at 03:23:23PM -0500, Pavel Roskin wrote: : On 16 Feb 2001, Alexandre Oliva wrote: : : > On Feb 16, 2001, Pavel Roskin <[EMAIL PROTECTED]> wrote: : > : > > I see. But we don't need reentrancy for configure, do we? : > : > Nope. Not even for config.status, AFAIK. We just need

Re: [patch] don't discard previous config.log before necessary

2001-02-17 Thread Pavel Roskin
Hello, Lars! > : I think it's an overkill. Errors during option parsing should go to > : stderr. This would solve the problem. > > The config.log file seems so wrong without those parts... Maybe you mean the parts that are generated _before_ option parsing? I can understand why they are importan

Re: [patch] remove config.status race condition

2001-02-17 Thread Pavel Roskin
Hello, Lars! I don't object against installing your patch as a temporary measure. Ideally, config.status should check one thing - that it's run by the same shell and on the same machine as configure - everything else must be hardcoded. But I realize that we have limited resources now, and that

Re: [patch] remove config.status race condition

2001-02-17 Thread Alexandre Oliva
On Feb 17, 2001, "Lars J. Aas" <[EMAIL PROTECTED]> wrote: > Configure-test concurrency will be introduced in Autoconf 3.0 :) :-) That, and sub-configure concurrency. I dream of `configure -j' every time I run configure on the so-called Cygnus unified tree... -- Alexandre Oliva Enjoy Guaran

Re: [patch] don't discard previous config.log before necessary

2001-02-17 Thread Alexandre Oliva
On Feb 13, 2001, "Lars J. Aas" <[EMAIL PROTECTED]> wrote: > +rm -f config.log > +mv AS_MESSAGE_LOG_NAME config.log > +m4_define([AS_MESSAGE_LOG_NAME], [config.log])dnl > +exec AS_MESSAGE_LOG_FD>>AS_MESSAGE_LOG_NAME]) On some systems, you can't rename an open file. Add `exec AS_MESSAGE_LOG_FD >&

Re: [patch] don't discard previous config.log before necessary

2001-02-17 Thread Lars J. Aas
On Sat, Feb 17, 2001 at 06:58:58PM -0200, Alexandre Oliva wrote: : On Feb 13, 2001, "Lars J. Aas" <[EMAIL PROTECTED]> wrote: : : > +rm -f config.log : > +mv AS_MESSAGE_LOG_NAME config.log : > +m4_define([AS_MESSAGE_LOG_NAME], [config.log])dnl : > +exec AS_MESSAGE_LOG_FD>>AS_MESSAGE_LOG_NAME]) :

Re: [patch] don't discard previous config.log before necessary

2001-02-17 Thread Alexandre Oliva
On Feb 17, 2001, "Lars J. Aas" <[EMAIL PROTECTED]> wrote: > : But I suppose we need some clean-up action for conf-log.tmp. > Yes. Do you know if configure has many exit points in AC_INIT (so a trap > should be set up) or if it's just the one after option parsing? I don't know :-( I lost track