zsh-workers
 help / color / mirror / code / Atom feed
From: "Peter A. Castro" <doctor@fruitbat.org>
To: zsh-workers@sunsite.dk
Subject: Re: zsh and line breaks
Date: Tue, 6 Apr 2004 15:25:23 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.53.0404061515260.11532@gremlin.fruitbat.org> (raw)
In-Reply-To: <12924.1081244614@csr.com>

On Tue, 6 Apr 2004, Peter Stephenson wrote:

> "Peter A. Castro" wrote:
> > Ok, see attached diff of the following files.  These are primarily Cygwin
> > specific fixes, but some could be considdered generic:
> >
> > Doc/Makefile.in - A little correction to the texi2html command
> >                   args and a fix to the install.html target to use sdir.
> > Src/builtin.c - Prevent from constructing "//path".
> > Src/exec.c    - change read to use actual length instead of stat'd length.
> > Src/main.c    - added Cygwin premain hook to deal with Text related I/O.
> > Src/system.h  - add #include of <sys/cygwin.h>.
> > configure     - change DLLD & DLLDFLAGS to not use dllwrap
> > configure.ac  - change DLLD & DLLDFLAGS to not use dllwrap
>
> Thanks, I've committed this apart from the texi2html hunk.

Thanks, Peter :)

> It's possible opening all files in text mode will cause problems
> elsewhere, e.g. on file descriptors passed to external programmes, but
> the text/binary problem is inescapable; we probably need to wait and
> see.  We can always invent grotesqueries like <#myfile.bin if it becomes
> necessary.

If it's any consolation, only files which are opened only as O_RDONLY are
massaged to be opened with the special O_TEXT attribute.  All O_WRONLY
and O_RDWR are left alone, which I hope, will produce the correct
semantic.  Since shells are primarily dealing with textual data I feel
this is a sufficient trade-off.  I will, of course, be following this
issue and report if there are any problems (which I'll probably have to
fix, since I opened this can'o'worms myself :)  But, as you say, time
will tell.

-- 
Peter A. Castro <doctor@fruitbat.org> or <Peter.Castro@oracle.com>
	"Cats are just autistic Dogs" -- Dr. Tony Attwood


  reply	other threads:[~2004-04-06 22:26 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <s38u10erwrz.fsf@ackusativ.ling.uu.se>
     [not found] ` <1vvf60l9i7n4nlobcroik2le5ofe6v2rn4@4ax.com>
     [not found]   ` <Pine.LNX.4.53.0403291441160.5939@gremlin.fruitbat.org>
2004-03-30  9:32     ` Oliver Kiddle
2004-03-30 18:54       ` Peter A. Castro
2004-03-31 12:45         ` Oliver Kiddle
2004-03-31 17:53           ` Peter A. Castro
2004-04-01  9:09             ` Oliver Kiddle
2004-04-01 21:20               ` Peter A. Castro
2004-04-02  9:49                 ` Dave Korn
2004-04-02 16:59                   ` Peter A. Castro
2004-04-03  1:23                 ` Peter A. Castro
2004-04-03 20:35                   ` Bart Schaefer
2004-04-04 19:29                     ` Peter Stephenson
2004-04-05 16:19                       ` Peter A. Castro
2004-04-05 17:40                         ` Peter Stephenson
2004-04-05 22:08                           ` Peter A. Castro
2004-04-05 23:08                             ` Clint Adams
2004-04-05 23:25                               ` Peter A. Castro
2004-04-07 21:10                                 ` Clint Adams
2004-04-08  1:16                                   ` Peter A. Castro
2004-04-09  5:00                                     ` Felix Rosencrantz
2004-04-09 16:26                                       ` Peter A. Castro
2004-04-06  9:43                             ` Peter Stephenson
2004-04-06 22:25                               ` Peter A. Castro [this message]
2004-04-15  5:03                             ` Clint Adams
2004-04-15  6:43                               ` Peter A. Castro
2004-04-15 10:35                                 ` Clint Adams
2004-04-15 16:57                                   ` Peter A. Castro
2004-04-15 18:14                                     ` Clint Adams
2005-01-31  5:07                             ` Patch for 4.2.3 building on Cygwin Peter A. Castro
2005-01-31 11:37                               ` Peter Stephenson
2005-01-31 20:48                                 ` Peter A. Castro
2005-02-01 10:53                                   ` Peter Stephenson
2005-02-01 16:03                                     ` Bart Schaefer
2005-02-01 18:04                                       ` Peter Stephenson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Pine.LNX.4.53.0404061515260.11532@gremlin.fruitbat.org \
    --to=doctor@fruitbat.org \
    --cc=zsh-workers@sunsite.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).