Eric Blake <e...@byu.net> writes:

> Eric Blake <ebb9 <at> byu.net> writes:
>
>> > +     -e 's/\r//g' |
>> 
>> I'm not sure that sed s/\r//g is portable.  You may have to resort to tr
>> instead.
>
> It's not (Solaris /bin/sed treats it like sed 's/\\r//g', looking for a 
> 2-byte 
> literal backslash and r sequence, rather than the intended 1-byte carriage 
> return).  Also, multiple sed -e is overkill when a single argument will do.  
> And seeing the transformations in reverse order just looked weird.  I'll be 
> committing this (probably munged by gmane, since it also fixes some 
> longer-than-
> 80-column lines):

Thank you!

/Simon


Reply via email to