#23927 [Opn->Bgs]: System::_parseArgv() fails if there are spaces

ID: 23927 Updated by: cellog@php.net Reported By: greg at chiaraquartet dot net -Status: Open +Status: Bogus Bug Type: PEAR related Operating System: Windows XP PHP Version: 4.3.2 New Comment: _parseArgv works if you pass it an array split along the correct places Previous Comments: ------------------------------------------------------------------------ [2003-07-27 05:52:00] arnaud@php.net what is preventing this patch from being applied ? It looks reasonnable to allow spaces in paths under windows. ------------------------------------------------------------------------ [2003-06-03 12:37:27] greg at chiaraquartet dot net this is not a bug in path, and is not a duplicate bug. Please read more carefully. this is a bug in command-line options. if you want to pass a value to a command-line option that has spaces in it, you can't. This is a valid bug. Please don't close this one. The PEAR packager is calling mktemp with the value returned from getting the cwd. Windows returns cwd with spaces in the pathname if they are there, you can complain to the authors of get_cwd about that, but it is the way PHP is written. I posted a fix that allows quoted attributes to be passed to _argv, it doesn't break BC, and fixes the problem, why mark this as bogus? ------------------------------------------------------------------------ [2003-06-03 12:28:21] pajoye@php.net Please do not submit the same bug more than once. An existing bug report already describes this very problem. Even if you feel that your issue is somewhat different, the resolution is likely to be the same. Because of this, we hope you add your comments to the existing bug instead. Thank you for your interest in PHP. Spaces are currently not allowed in path, you may use the short names under win32 (c:/progra~1). There is BC issue to take care about this problem. btw, System::_parseArgv() is a private method, I hope you do not rely on it :) pierre ------------------------------------------------------------------------ [2003-05-31 17:34:57] greg at chiaraquartet dot net In addition, line 382 of System.php (in System::mktemp()) needs to be: if (!System::mkDir("-p \"$tmpdir\"")) { instead of if (!System::mkDir("-p $tmpdir")) { after the last applied patch ------------------------------------------------------------------------ [2003-05-31 16:27:55] greg at chiaraquartet dot net A fix allows quoted arguments, so it would be called with -t "C:\Web Pages" -d and expand properly. Here is the diff with System.php: 64a65,93 > $newargv = array(); > $in_quote = false; > $index = 0; > foreach($argv as $arg) > { > if (!isset($newargv[$index])) { > $newargv[$index] = ''; > } > // combine quoted values into 1 string > if (!$in_quote && $arg{0} == '"') { > $newargv[$index] = substr($arg,1); > $in_quote = true; > } elseif ($in_quote && $arg{strlen($arg) - 1} == '"') { > $arg = ' ' . $arg; > $newargv[$index] .= substr($arg,0,strlen($arg) - 1); > $in_quote = false; > $index++; > } else { > if ($in_quote) { > $arg .= ' '; > } > $newargv[$index] .= $arg; > if (!$in_quote) { > // go to next value if not in a quote > $index++; > } > } > } > $argv = $newargv; and the diff against Packager.php: 119c119 < if (!($tmpdir = System::mktemp('-t '.getcwd().' -d'))) { --- > if (!($tmpdir = System::mktemp('-t "'.getcwd().'" -d'))) > { ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/23927 -- Edit this bug report at http://bugs.php.net/?id=23927&edit=1