diff options
| -rw-r--r-- | ChangeLog | 7 | ||||
| -rw-r--r-- | win32/win32.c | 17 |
2 files changed, 18 insertions, 6 deletions
@@ -1,3 +1,10 @@ +Sat Sep 6 01:23:22 2003 NAKAMURA Usaku <usa@ruby-lang.org> + + * win32/win32.c (CreateChild): need to quote cmd if RUBYSHELL is set. + + * win32/win32.c (CreateChild): fix condition about whether to call + shell or not. + Sat Sep 6 00:36:20 2003 Nobuyoshi Nakada <nobu.nokada@softhome.net> * Makefile.in (test): phony target. diff --git a/win32/win32.c b/win32/win32.c index b7827218ad..0e75f431a1 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -910,23 +910,28 @@ CreateChild(char *cmd, char *prog, SECURITY_ATTRIBUTES *psa, HANDLE hInput, HAND dwCreationFlags = (NORMAL_PRIORITY_CLASS); - shell = NULL; if (prog) { shell = prog; } - else if (has_redirection(cmd)) { - if (shell = getenv("RUBYSHELL")) { + else { + int redir = -1; + if ((shell = getenv("RUBYSHELL")) && (redir = has_redirection(cmd))) { char *tmp = ALLOCA_N(char, strlen(shell) + strlen(cmd) + - sizeof (" -c ")); - sprintf(tmp, "%s -c %s", shell, cmd); + sizeof (" -c ") + 2); + sprintf(tmp, "%s -c \"%s\"", shell, cmd); cmd = tmp; } - else if ((shell = getenv("COMSPEC")) && isInternalCmd(cmd, shell)) { + else if ((shell = getenv("COMSPEC")) && + ((redir < 0 ? has_redirection(cmd) : redir) || + isInternalCmd(cmd, shell))) { char *tmp = ALLOCA_N(char, strlen(shell) + strlen(cmd) + sizeof (" /c ")); sprintf(tmp, "%s /c %s", shell, cmd); cmd = tmp; } + else { + shell = NULL; + } } RUBY_CRITICAL({ |