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'))) {

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

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

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?

what is preventing this patch from being applied ? It looks reasonnable to allow spaces in paths under windows.

_parseArgv works if you pass it an array split along the correct places