← Back to index | Original Bugzilla link

Bug 13447 – Do not escape process parameters unless necessary

Status: RESOLVED
Resolution: FIXED
Severity: normal
Priority: P1
Component: phobos
Product: D
Version: D2
Platform: All
OS: Windows
Creation time: 2014-09-09T14:36:00Z
Last change time: 2014-09-09T16:04:12Z
Keywords: pull
Assigned to: nobody
Creator: dlang-bugzilla

Comments

Comment #0 by dlang-bugzilla — 2014-09-09T14:36:20Z

Some Windows programs do not use CommandLineToArgvW. Examples: WinMerge, GoLink, older programs written for Windows versions before Windows 2000 (which introduced CommandLineToArgvW). These programs can't handle switches when they are enclosed in quotes (e.g. "/r"). std.process should not quote parameters unless it is actually necessary for them to be parsed correctly.

Comment #1 by dlang-bugzilla — 2014-09-09T14:38:50Z

https://github.com/D-Programming-Language/phobos/pull/2504

Comment #2 by github-bugzilla — 2014-09-09T16:04:11Z

Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/09a0b876c87cec6bb7af62b55179aff745b203dc fix Issue 13447 - Do not escape process parameters unless necessary https://github.com/D-Programming-Language/phobos/commit/f35c21972fefc27e1f88f43eb577221e31511f25 Merge pull request #2504 from CyberShadow/pull-20140909-143654 fix Issue 13447 - Do not escape process parameters unless necessary