From 6fa6a1f865b56b1dee05d780c788686db3bd3c2a Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Tue, 6 Apr 2004 09:25:17 +0000 Subject: Peter Castro: Cygwin improvements for dynamic libraries, text/binary, paths --- Src/builtin.c | 10 ++++++++++ Src/exec.c | 7 ++++--- Src/main.c | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Src/system.h | 1 + 4 files changed, 71 insertions(+), 3 deletions(-) (limited to 'Src') diff --git a/Src/builtin.c b/Src/builtin.c index 2ae7e9bbf..8703ecead 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -927,7 +927,9 @@ cd_do_chdir(char *cnam, char *dest, int hard) * DOS style names with drives in them */ static char buf[PATH_MAX]; +#ifndef _SYS_CYGWIN_H void cygwin_conv_to_posix_path(const char *, char *); +#endif cygwin_conv_to_posix_path(dest, buf); dest = buf; @@ -1031,7 +1033,15 @@ cd_try_chdir(char *pfix, char *dest, int hard) /* handle directory prefix */ if (pfix && *pfix) { if (*pfix == '/') +#ifdef __CYGWIN__ +/* NB: Don't turn "/"+"bin" into "//"+"bin" by mistake! "//bin" may * + * not be what user really wants (probably wants "/bin"), but * + * "//bin" could be valid too (see fixdir())! This is primarily for * + * handling CDPATH correctly. */ + buf = tricat(pfix, ( pfix[1] == '\0' ? "" : "/" ), dest); +#else buf = tricat(pfix, "/", dest); +#endif else { int pfl = strlen(pfix); dlen = strlen(pwd); diff --git a/Src/exec.c b/Src/exec.c index 4bd6503bf..8256910b3 100644 --- a/Src/exec.c +++ b/Src/exec.c @@ -3662,6 +3662,7 @@ getfpfunc(char *s, int *ksh) { char **pp, buf[PATH_MAX]; off_t len; + off_t rlen; char *d; Eprog r; int fd; @@ -3681,12 +3682,12 @@ getfpfunc(char *s, int *ksh) if ((len = lseek(fd, 0, 2)) != -1) { d = (char *) zalloc(len + 1); lseek(fd, 0, 0); - if (read(fd, d, len) == len) { + if ((rlen = read(fd, d, len)) >= 0) { char *oldscriptname = scriptname; close(fd); - d[len] = '\0'; - d = metafy(d, len, META_REALLOC); + d[rlen] = '\0'; + d = metafy(d, rlen, META_REALLOC); scriptname = dupstring(s); r = parse_string(d); diff --git a/Src/main.c b/Src/main.c index c9c5fa867..30eef5a25 100644 --- a/Src/main.c +++ b/Src/main.c @@ -30,6 +30,62 @@ #include "zsh.mdh" #include "main.pro" +/* + * Support for Cygwin binary/text mode filesystems. + * Peter A. Castro + * + * This deserves some explaination, because it uses Cygwin specific + * runtime functions. + * + * Cygwin supports the notion of binary or text mode access to files + * based on the mount attributes of the filesystem. If a file is on + * a binary mounted filesystem, you get exactly what's in the file, CRLF's + * and all. If it's on a text mounted filesystem, Cygwin will strip out + * the CRs. This presents a problem because zsh code doesn't allow for + * CRLF's as line terminators. So, we must force all open files to be + * in text mode reguardless of the underlying filesystem attributes. + * However, we only want to do this for reading, not writing as we still + * want to write files in the mode of the filesystem. To do this, + * we have two options: augment all {f}open() calls to have O_TEXT added to + * the list of file mode options, or have the Cygwin runtime do it for us. + * I choose the latter. :) + * + * Cygwin's runtime provides pre-execution hooks which allow you to set + * various attributes for the process which effect how the process functions. + * One of these attributes controls how files are opened. I've set + * it up so that all files opened RDONLY will have the O_TEXT option set, + * thus forcing line termination manipulation. This seems to solve the + * problem (at least the Test suite runs clean :). + * + * Note: this may not work in later implementations. This will override + * all mode options passed into open(). Cygwin (really Windows) doesn't + * support all that much in options, so for now this is OK, but later on + * it may not, in which case O_TEXT will have to be added to all opens calls + * appropriately. + * + * This function is actually a hook in the Cygwin runtime which + * is called before the main of a program. Because it's part of the program + * pre-startup, it must be located in the program main and not in a DLL. + * It must also be made an export so the linker resolves this function to + * our code instead of the default Cygwin stub routine. + */ + +/**/ +#ifdef __CYGWIN__ +/**/ +mod_export void +cygwin_premain0 (int argc, char **argv, void *myself) +{ + static struct __cygwin_perfile pf[] = + { + {"", O_RDONLY | O_TEXT}, + {NULL, 0} + }; + cygwin_internal (CW_PERFILE, pf); +} +/**/ +#endif /* __CYGWIN__ */ + /**/ int main(int argc, char **argv) diff --git a/Src/system.h b/Src/system.h index 558508d2d..f03d890c4 100644 --- a/Src/system.h +++ b/Src/system.h @@ -675,6 +675,7 @@ extern short ospeed; #endif #ifdef __CYGWIN__ +# include # define IS_DIRSEP(c) ((c) == '/' || (c) == '\\') #else # define IS_DIRSEP(c) ((c) == '/') -- cgit 1.4.1