[ANNOUNCEMENT] Updated: bash-2.05b-4

[dvasaru@broadpark.no]

Quoting Corinna Vinschen <corinna-cygwin@cygwin.com>:

> On Tue, Sep 10, 2002 at 08:01:00AM +0200, Dan Vasaru wrote:
> > I'll give this a go in gdb as it seems to have caused problems for some
> > users. Did you just remove the fd_is_seekable ifdef  ?
> Yes, I announced -5 yesterday, reverting that again.

The problem is with the way text seeks are (not) implemented in cygwin: lseek in text files do not adjust for skipped CRs. This is also the strategy MS is using in their CRT, so it may be the most MSVCRT/CYGWIN compatible solution.

So it seems that applications that use lseek on text files should handle CRLF translation by themselves, at least for now.

In the case of bash, CRLF translation seems to have been implemented by the DJGPP port.

I just removed (almost) all __CYGWIN__  textmode translation related #ifdefs, and enabled the DJGPP CR removal hack in buffered_getchar ().

Preliminary testing shows the new bash to work on both CRLF and LF files.
Files are no longer opened in O_TEXT mode, but CR's are stripped in buffered_getchar.
In case anyone is interested in testing out the new bash, I'm attaching a diff to bash-2.05b-3-src.tar.bz2, and I can provide a prebuilt bash.exe.

dan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bashdiff
Type: application/octet-stream
Size: 2380 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin/attachments/20020910/5b31e123/attachment.obj>
-------------- next part --------------
--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/