Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-08 Thread Stefano Lattarini
On 02/08/2012 03:19 PM, Peter Rosin wrote: > > Thanks! > > Pushed, as Eric already seemed in favor of keeping scriptversion linear. > Yes, I've stalled the "$scriptversion removal plan" for the moment. And anyway, that plan shouldn't have affected any of your patches: I would have implemented it

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-08 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-08 15:07: > On 02/08/2012 02:58 PM, Jim Meyering wrote: >> Peter Rosin wrote: >>> This thread has stalled. To sum things up and get things going again, >>> the following is what I'd like to do. First commit the three patches >>> (that are replies to this message) o

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-08 Thread Stefano Lattarini
On 02/08/2012 02:58 PM, Jim Meyering wrote: > Peter Rosin wrote: >> This thread has stalled. To sum things up and get things going again, >> the following is what I'd like to do. First commit the three patches >> (that are replies to this message) on the msvc branch. Then merge msvc >> into mast

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-08 Thread Jim Meyering
Peter Rosin wrote: > This thread has stalled. To sum things up and get things going again, > the following is what I'd like to do. First commit the three patches > (that are replies to this message) on the msvc branch. Then merge msvc > into master and into branch-1.11. > > 2/3 and 3/3 are alrea

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-08 Thread Peter Rosin
Hi! This thread has stalled. To sum things up and get things going again, the following is what I'd like to do. First commit the three patches (that are replies to this message) on the msvc branch. Then merge msvc into master and into branch-1.11. 2/3 and 3/3 are already ACKed by Stefano and s

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Eric Blake skrev 2012-02-03 19:42: > On 02/03/2012 11:29 AM, Peter Rosin wrote: >> Jim Meyering skrev 2012-02-03 19:05: >>> Stefano Lattarini wrote: On 02/03/2012 03:51 PM, Peter Rosin wrote: > Wait, there's a (trivial) conflict. But if I resolve that and go > on with the merge anyway

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-03 18:41: > On 02/03/2012 03:51 PM, Peter Rosin wrote: >> Maybe depmod.tap should be replaced/rewritten with "compilers" that simulate the different depmods? I could tinker with that a bit... >>> Yeah, I had thought about the possibility of such an ap

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Jim Meyering
Eric Blake wrote: > On 02/03/2012 11:29 AM, Peter Rosin wrote: >> Jim Meyering skrev 2012-02-03 19:05: >>> Stefano Lattarini wrote: On 02/03/2012 03:51 PM, Peter Rosin wrote: > Wait, there's a (trivial) conflict. But if I resolve that and go > on with the merge anyway we will end up w

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Eric Blake
On 02/03/2012 11:29 AM, Peter Rosin wrote: > Jim Meyering skrev 2012-02-03 19:05: >> Stefano Lattarini wrote: >>> On 02/03/2012 03:51 PM, Peter Rosin wrote: Wait, there's a (trivial) conflict. But if I resolve that and go on with the merge anyway we will end up with two different depcomp

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Jim Meyering skrev 2012-02-03 19:05: > Stefano Lattarini wrote: >> On 02/03/2012 03:51 PM, Peter Rosin wrote: >>> Wait, there's a (trivial) conflict. But if I resolve that and go >>> on with the merge anyway we will end up with two different depcomps >>> with "scriptversion=2012-02-03.12; # UTC".

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Jim Meyering
Stefano Lattarini wrote: > On 02/03/2012 03:51 PM, Peter Rosin wrote: >> Maybe depmod.tap should be replaced/rewritten with "compilers" that simulate the different depmods? I could tinker with that a bit... >>> Yeah, I had thought about the possibility of such an approach, but was >

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Stefano Lattarini
On 02/03/2012 03:51 PM, Peter Rosin wrote: > >>> Maybe depmod.tap should be replaced/rewritten with "compilers" that >>> simulate the different depmods? I could tinker with that a bit... >>> >> Yeah, I had thought about the possibility of such an approach, but was >> reluctant to suggest it, sinc

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-03 15:03: > On 02/03/2012 02:24 PM, Peter Rosin wrote: >> Stefano Lattarini skrev 2012-02-03 13:32: >>> On 02/03/2012 10:07 AM, Peter Rosin wrote: >>> --- a/lib/depcomp +++ b/lib/depcomp @@ -1,7 +1,7 @@ #! /bin/sh # depcomp - compile a prog

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Stefano Lattarini
On 02/03/2012 02:24 PM, Peter Rosin wrote: > Stefano Lattarini skrev 2012-02-03 13:32: >> On 02/03/2012 10:07 AM, Peter Rosin wrote: >> >>> --- a/lib/depcomp >>> +++ b/lib/depcomp >>> @@ -1,7 +1,7 @@ >>> #! /bin/sh >>> # depcomp - compile a program generating dependencies as side-effects >>> >>

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-03 13:32: > On 02/03/2012 10:07 AM, Peter Rosin wrote: >> Stefano Lattarini skrev 2012-02-03 09:35: *snip* >> I'd rather do that in a follow-up, like below, so that the real change >> in the above patch isn't hidden in the churn. >> > Sounds sensible; agreed. > >>

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Stefano Lattarini
On 02/03/2012 10:07 AM, Peter Rosin wrote: > Stefano Lattarini skrev 2012-02-03 09:35: >> Hi Peter. >> >> On 02/03/2012 08:58 AM, Peter Rosin wrote: >>> Commit Release-1-7-2b-2-gf03ceab "Cope with DOS filenames in >>> dependencies." inadvertently converted tabs into spaces. >>> >>> * lib/depcomp (d

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-03 09:35: > Hi Peter. > > On 02/03/2012 08:58 AM, Peter Rosin wrote: >> Commit Release-1-7-2b-2-gf03ceab "Cope with DOS filenames in >> dependencies." inadvertently converted tabs into spaces. >> >> * lib/depcomp (dashmstdout): Add a tab character to all sets >> mat

Re: [PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-03 Thread Stefano Lattarini
Hi Peter. On 02/03/2012 08:58 AM, Peter Rosin wrote: > Commit Release-1-7-2b-2-gf03ceab "Cope with DOS filenames in > dependencies." inadvertently converted tabs into spaces. > > * lib/depcomp (dashmstdout): Add a tab character to all sets > matching whitespace. > --- > lib/depcomp |6 +++---

[PATCH] depcomp: recognize tabs as whitespace in the dashmstdout mode

2012-02-02 Thread Peter Rosin
Commit Release-1-7-2b-2-gf03ceab "Cope with DOS filenames in dependencies." inadvertently converted tabs into spaces. * lib/depcomp (dashmstdout): Add a tab character to all sets matching whitespace. --- lib/depcomp |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) I know the Tru6