Description: ------------ A simple command executed in Windows is broken by the "fix" included in the SVN.php at line 654 is actually preventing VersionControl_SVN from correctly executing the wanted command. Expected result: ---------------- No errors generated. Actual result: -------------- "svn log --username REMOVED --password REMOVED --no-auth-cache --verbose --non-interactive --xml "C:' is not recognized as an internal or external command, (cmd: "svn log --username REMOVED --password REMOVED --no-auth-cache --verbose --non-interactive --xml "C:/Complex Path/Local Copy/"")

The issues is that it puts the quotation marks around the whole command instead of only around the executable path. Ie: "C:\Program Files\Subversion\bin\svn.exe log http://localhost/repos/test" should be: "C:\Program Files\Subversion\bin\svn.exe" log http://localhost/repos/test

I made a temporary fix for myself, replace: $cmd = '"' . $cmd . '"'; with: $cmd = str_replace( $this->svn_path, '"'.$this->svn_path.'"', $cmd );

This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.