On Saturday 18 December 2010, Ralf Wildenhues wrote:
> [MEGA CUT]
> > > > > >  # Don't redefine several times the same variable.
> > > > > > -cp Makefile.am Makefile.src
> > > > > > +cp -f Makefile.am Makefile.src
> > > > > 
> > > > > Why should you need this change?  Generally, I don't see why you ever
> > > > > need 'cp -f'.
> > > > >
> > > > I dimly remember that I used to think there were cp implementations 
> > > > which
> > > > might prompt if stdout/stderr is a tty and the dest file exists, unless
> > > > the `-f' option is used..
> > > 
> > > That is true, but when you run a test, there is no tty involved.
> 
> That part (about the tty) is not true.  Sorry about that.
> 
OK, noted.

> > Even when I run it from the shell prompt?
> 
> cp should only ever prompt when given -i.  That may happen through a
> shell alias.  But aliases are not normally enabled in scripts.
>
OK, then I'll keep the `cp -f' change reverted.

> If and when we find a cp that does prompt from a script,
>
Let's hope we never do.

> then we can think of going back to this; but only after amending
> autoconf.texi (which, by the way, also documents that some old cp 
> versions do not accept -f at all).
> 
I've now merged the patch into master, and pushed.

Thanks,
   Stefano

Reply via email to