[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[FYI] pushed to master (was: Re: [PATCH] {master} Extend, fix and improv
From: |
Stefano Lattarini |
Subject: |
[FYI] pushed to master (was: Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support.) |
Date: |
Sat, 18 Dec 2010 11:59:59 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
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
- [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Stefano Lattarini, 2010/12/13
- Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Ralf Wildenhues, 2010/12/16
- Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Stefano Lattarini, 2010/12/16
- Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Ralf Wildenhues, 2010/12/17
- Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Stefano Lattarini, 2010/12/17
- Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support., Ralf Wildenhues, 2010/12/18
- [FYI] pushed to master (was: Re: [PATCH] {master} Extend, fix and improve tests on Lex and Yacc support.),
Stefano Lattarini <=