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

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

 



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)?

Or is your rationale "this will prevent people from screwing up the test
scripts accidentally in the future"?

I am not opposed to the latter, but I think it makes sense to state it
clearly in the commit message.

-Peff
--
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