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

[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 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> 
> I know the Tru64 patch to depcomp is out there, and will wait with
> pushing this one until that one has been processed, but I just wanted
> to get this out...
>
Don't worry about that patch, your objection to how the depcomp.tap
test is written makes a lot of sense (unfortunately for me), so I now
think the right fix for bug #10434 will be to change the test case
instead of the depcomp script.

> Ok for msvc?
>
Almost...  to avoid similar regressions in the future, I think we
could introduce a new '$tab' variable and use that instead...  and
then, for extra safety, we might even add a sanity check like:

  case "$tab" in *\ *) fatal "\$tab is not a TAB";; esac

early in the script, to ensure $tab is not messed up by editors or
plain old carelessness.

WDYT?

Thanks,
  Stefano