Jeff King, Thu, Jun 12, 2008 00:54:48 +0200: > On Wed, Jun 11, 2008 at 08:25:01PM +0200, Alex Riesen wrote: > > > It offers a limited set of POSIX tr, in particular: no character class > > support and no [n*m] operators. Only 8bit. C-escapes supported, and > > character ranges. Deletion and squeezing should work, but -s does not > > match the GNU tr from coreutils (which, in turn, does not match POSIX). > > > > Signed-off-by: Alex Riesen <raa.lkml@xxxxxxxxx> > > --- > > > > Rebased on top of current master. I still think it makes the test > > suite more portable. > > Having wrestled with Solaris tr, I can understand where you are coming > from. However, does this _actually_ increase the portability of the test > suite? That is, are there failing tests that this fixes, and if so, for > which platforms (getting a successful run of the test suite on Solaris > is still on my long-term todo, but I thought I had fixed all of the tr > issues)? 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). 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?). > Or is your rationale "this will prevent people from screwing up the test > scripts accidentally in the future"? We just can't have that. Nothing can prevent people from screwing up anything in any given point of time :) 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). And maybe someday my employment situation improves and I wont push it at all :) -- 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
