Re: sed strips CRs

2012-02-15 Thread Corinna Vinschen
On Feb 15 01:13, Andrey Repin wrote: > Greetings, Earnie Boyd! > > >>> The standard response to issues dealing with CRLF files is to point the > >>> user to dos2unix and text mode mounts. This should be adequate without > >>> the hidden behavior of sed/grep/awk and probably others. > >> > >> While

Re: sed strips CRs

2012-02-14 Thread Andrey Repin
Greetings, Earnie Boyd! >>> The standard response to issues dealing with CRLF files is to point the >>> user to dos2unix and text mode mounts. This should be adequate without >>> the hidden behavior of sed/grep/awk and probably others. >> >> While your reasoning is sound, I prefer them to behave t

Re: sed strips CRs

2012-02-14 Thread Earnie Boyd
On Mon, Feb 13, 2012 at 7:56 PM, Andrey Repin wrote: > Greetings, Nellis, Kenneth! > >> The standard response to issues dealing with CRLF files is to point the >> user to dos2unix and text mode mounts. This should be adequate without >> the hidden behavior of sed/grep/awk and probably others. > > W

Re: sed strips CRs

2012-02-13 Thread Andrey Repin
Greetings, Nellis, Kenneth! > The standard response to issues dealing with CRLF files is to point the > user to dos2unix and text mode mounts. This should be adequate without > the hidden behavior of sed/grep/awk and probably others. While your reasoning is sound, I prefer them to behave the way

Re: sed strips CRs

2012-02-13 Thread Earnie Boyd
On Mon, Feb 13, 2012 at 2:48 PM, Paolo Bonzini wrote: > > If you meant that "rt" should be restricted to cygwin, that's also fine by > me but in general I prefer feature tests to OS tests. > Then it becomes Cygwin's problem. I'm going to quote from http://msdn.microsoft.com/en-us/library/yeby3zcb

Re: sed strips CRs

2012-02-13 Thread Paolo Bonzini
On 02/13/2012 08:42 PM, John Cowan wrote: > By the way, I'm still opening the script file with "rt". I cannot think > of any case when you would want to keep CRs there. You wouldn't, but the point is that "rt" isn't defined on Posix systems. If it happens to be the same as "r", good, but that i

Re: sed strips CRs

2012-02-13 Thread John Cowan
Paolo Bonzini scripsit: > By the way, I'm still opening the script file with "rt". I cannot think > of any case when you would want to keep CRs there. You wouldn't, but the point is that "rt" isn't defined on Posix systems. If it happens to be the same as "r", good, but that isn't guaranteed.

Re: sed strips CRs

2012-02-13 Thread Paolo Bonzini
On 02/13/2012 04:43 PM, Earnie Boyd wrote: > > By the way, I'm still opening the script file with "rt". I cannot think of > any case when you would want to keep CRs there. The case of sed -e 's/something/nothing/g' myfile > myfile2 as it works in Cygwin today would mean that in the case of th

Re: sed strips CRs

2012-02-13 Thread Peter Rosin
Earnie Boyd skrev 2012-02-13 16:43: > On Mon, Feb 13, 2012 at 10:22 AM, Paolo Bonzini wrote: >> On 02/13/2012 03:56 PM, Corinna Vinschen wrote: >>> As long as it's consistent with coreutils I'll certainly do the change. >>> >>> Thanks! Would you mind to CC the cygwin list when the next upstr

Re: sed strips CRs

2012-02-13 Thread Earnie Boyd
On Mon, Feb 13, 2012 at 10:22 AM, Paolo Bonzini wrote: > On 02/13/2012 03:56 PM, Corinna Vinschen wrote: >> >> > As long as it's consistent with coreutils I'll certainly do the change. >> >> Thanks!  Would you mind to CC the cygwin list when the next upstream >> sed release is available? > > > Sur

Re: sed strips CRs

2012-02-13 Thread Corinna Vinschen
On Feb 13 16:22, Paolo Bonzini wrote: > On 02/13/2012 03:56 PM, Corinna Vinschen wrote: > >> As long as it's consistent with coreutils I'll certainly do the change. > > > >Thanks! Would you mind to CC the cygwin list when the next upstream > >sed release is available? > > Sure, it should be real

Re: sed strips CRs

2012-02-13 Thread Paolo Bonzini
On 02/13/2012 03:56 PM, Corinna Vinschen wrote: > As long as it's consistent with coreutils I'll certainly do the change. Thanks! Would you mind to CC the cygwin list when the next upstream sed release is available? Sure, it should be real soon now since a new release has been long overdue.

Re: sed strips CRs

2012-02-13 Thread Corinna Vinschen
[Sent again. I missed all the CC's in my previous reply. Sorry!] On Feb 13 15:37, Paolo Bonzini wrote: > On 02/13/2012 03:12 PM, Eric Blake wrote: > >But fixing this should be done upstream, and not in cygwin. > > As long as it's consistent with coreutils I'll certainly do the change. > > Paol

Re: sed strips CRs

2012-02-13 Thread Corinna Vinschen
On Feb 13 15:37, Paolo Bonzini wrote: > On 02/13/2012 03:12 PM, Eric Blake wrote: > >But fixing this should be done upstream, and not in cygwin. > > As long as it's consistent with coreutils I'll certainly do the change. > > Paolo Thanks! Would you mind to CC the cygwin list when the next upstr

RE: sed strips CRs

2012-02-13 Thread Nellis, Kenneth
From: Earnie Boyd >On Sat, Feb 11, 2012 at 12:56 PM, Corinna Vinschen wrote:> >> That's why I wrote "on systems supporting this mode".  Sed input is >> text input in the first place.  Therefore it's using textmode in the >> first place.  This is done so for a long time.  If it's not what you >> nee

Re: sed strips CRs

2012-02-13 Thread Paolo Bonzini
On 02/13/2012 03:12 PM, Eric Blake wrote: But fixing this should be done upstream, and not in cygwin. As long as it's consistent with coreutils I'll certainly do the change. Paolo -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documenta

Re: sed strips CRs

2012-02-13 Thread Earnie Boyd
On Mon, Feb 13, 2012 at 9:12 AM, Eric Blake wrote: > > Personally, I think it is a bug that upstream sed is using 't' in > fopen() in the first place.  Linux does NOT have an 'rt' mode for a > reason: 't' is non-standard.  On cygwin, the preference used in > coreutils is that you get text mode by u

Re: sed strips CRs

2012-02-13 Thread Eric Blake
[adding bug-sed - see this thread in cygwin: http://cygwin.com/ml/cygwin/2012-02/msg00313.html] On 02/11/2012 10:19 AM, Earnie Boyd wrote: >>> By this I assume you to mean that the -b option opens the input file >>> in binary mode. But the mount table the OP showed was already in >>> binary mode.

Re: sed strips CRs

2012-02-13 Thread Earnie Boyd
On Sat, Feb 11, 2012 at 12:56 PM, Corinna Vinschen wrote: > That's why I wrote "on systems supporting this mode".  Sed input is > text input in the first place.  Therefore it's using textmode in the > first place.  This is done so for a long time.  If it's not what you > need, there's a workaround,

Re: sed strips CRs

2012-02-11 Thread Corinna Vinschen
On Feb 11 12:19, Earnie Boyd wrote: > On Sat, Feb 11, 2012 at 5:06 AM, Corinna Vinschen wrote: > > On Feb 10 14:44, Earnie Boyd wrote: > >> On Fri, Feb 10, 2012 at 10:07 AM, Corinna Vinschen wrote: > >> > On Feb 10 08:02, Nellis, Kenneth wrote: > >> >> Cygwin 'sed' seems to be stripping CRs from it

Re: sed strips CRs

2012-02-11 Thread Earnie Boyd
On Sat, Feb 11, 2012 at 5:06 AM, Corinna Vinschen wrote: > On Feb 10 14:44, Earnie Boyd wrote: >> On Fri, Feb 10, 2012 at 10:07 AM, Corinna Vinschen wrote: >> > On Feb 10 08:02, Nellis, Kenneth wrote: >> >> Cygwin 'sed' seems to be stripping CRs from its input. >> >> Linux sed doesn't do this. Exam

Re: sed strips CRs

2012-02-11 Thread Corinna Vinschen
On Feb 10 14:44, Earnie Boyd wrote: > On Fri, Feb 10, 2012 at 10:07 AM, Corinna Vinschen wrote: > > On Feb 10 08:02, Nellis, Kenneth wrote: > >> Cygwin 'sed' seems to be stripping CRs from its input. > >> Linux sed doesn't do this. Example: > > > > Try the -b option. > > By this I assume you to me

Re: sed strips CRs

2012-02-10 Thread Earnie Boyd
On Fri, Feb 10, 2012 at 10:07 AM, Corinna Vinschen wrote: > On Feb 10 08:02, Nellis, Kenneth wrote: >> Cygwin 'sed' seems to be stripping CRs from its input. >> Linux sed doesn't do this. Example: > > Try the -b option. By this I assume you to mean that the -b option opens the input file in binary

RE: sed strips CRs

2012-02-10 Thread Nellis, Kenneth
From: Corinna Vinschen > On Feb 10 08:02, Nellis, Kenneth wrote: > > Cygwin 'sed' seems to be stripping CRs from its input. > > Linux sed doesn't do this. Example: > > Try the -b option. Thanx for that. It's not a problem for me. I was just reporting what looks like a Cygwin bug in case someo

Re: sed strips CRs

2012-02-10 Thread Corinna Vinschen
On Feb 10 08:02, Nellis, Kenneth wrote: > Cygwin 'sed' seems to be stripping CRs from its input. > Linux sed doesn't do this. Example: Try the -b option. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader cygwin AT cygwin DOT