Re: [PATCH] Add test-tr: poor-man tr

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jeff King, Thu, Jun 12, 2008 08:23:09 +0200:
> On Thu, Jun 12, 2008 at 08:01:52AM +0200, Alex Riesen wrote:
> 
> > Frankly, it started because I wanted to minimize use of Perl on
> > Windows (because I can't get around ActiveState Perl at work, and it
> > breaks almost everything it touches). Accidentally, it is also faster
> > there (maybe just because it's smaller).
> 
> Ah, right. Well, I am not opposed to getting rid of perl in the test
> scripts (there is core functionality provided by perl, so one can easily
> run git on a system with no perl at all).

"git add --interactive" and "git send-mail". The first has a very good
replacement (git gui). The other, even if not that "core", is a very
popular tool. (And I can't use it at work at all: noone to understand
word "patch" there. And even then, noone would care about change
review anyway).

> However, even with your patch, there is still some perl left, so I am
> not sure that it has really bought us very much.

I was forced to carefully check every instance and am positive that
the others are ok with regard to ActiveState Perl and Windows quirks :)
(IOW, I run the testsuite every day and it aint broke yet)

> > But, as was already noted, tr does not behave the same for all
> > platforms (there were even differences in output, BSD or Solaris put
> > out a stray LF?).
> 
> I think those were all resolved by using perl, and your patch replaces
> them with test-tr.

That was the problem. ActiveState Perl always replaces LF in the
output with CRLF, which caused mismatches with template files in some
tests (even the generated templates had LF line endings, cygwins tools
follow that convention). At first, I tried to get by putting
"binmode(STDOUT)" into every test, but this became boring with a time.
Besides, the lines get very long and ugly (and make conflict resolving
harder).

> > In any case, I wont push this change too hard. I must admit, that
> > there is no real good reason besides one "screwed" company using
> > obsoleted tools in a weird way. And it is a maintenance effort (and
> > people will forget to use test-tr instead of perl and tr).
> 
> There is maintenance effort either way; people need to know not to do
> unportable things with tr (and other tools), and the solution to that is
> to run the test scripts on each platform (something we are starting to
> do).

With test-tr they still have to do all that, but also support its
existence. So I'm not sure.

> So I am not opposed to test-tr, I just wanted you to explain it better
> in the commit log. ;)

I already tried.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux