This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: Fwd: Re: more coreutils 5.3.0 issues on cygwin
- From: Corinna Vinschen <corinna-cygwin at cygwin dot com>
- To: cygwin at cygwin dot com
- Date: Tue, 18 Jan 2005 15:23:56 +0100
- Subject: Re: Fwd: Re: more coreutils 5.3.0 issues on cygwin
- References: <41ED16DB.3020701@byu.net>
- Reply-to: cygwin at cygwin dot com
On Jan 18 07:02, Eric Blake wrote:
> Corinna, why did you change od in coreutils 5.2.1? When I proposed your
> patch upstream, the maintainers pointed out that 12 lines later in the
> same method, the code is calling setmode(fd, O_BINARY) for all non-tty
> input filenames that were just fopen'd. Does cygwin have a working
> setmode (and if not, does fcntl(fd, F_SETFL, O_BINARY | fcntl(fd,
> F_GETFL)) work as expected)?
He's of course right. Just keep the patch in as a Cygwin local patch
which shouldn't go upstream. What happened was that od failed to read
files in binary mode on textmode mounts. It turned out that recent
Cygwin/newlib changes resulted in setmode only working correctly on
the stdio descriptors 0, 1 and 2. This is fixed in CVS, but the
problem persists in 1.5.12. The patch circumvents that problem in
1.5.12, but it will become unnecessary later on.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader mailto:cygwin@cygwin.com
Red Hat, Inc.
--
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
Problem reports: http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ: http://cygwin.com/faq/