[Scons-dev] Issue 1689's solution no longer needed, removing would fix 2225

Bill Deegan bill at baddogconsulting.com
Sun Feb 28 21:00:06 EST 2016


Greetings,

It seems to me that the patch to resolve
http://scons.tigris.org/issues/show_bug.cgi?id=1689
(adding parens to the list of "special" characters to be shell escaped)

Is no longer needed since the switch to subprocess.

Thoughts?
Am I off base here?
Rolling back the addition of parens fixes bug:
http://scons.tigris.org/issues/show_bug.cgi?id=2225

I also have created a unit testcase to check that parens are no longer
shell escaped.

-Bill
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist2.pair.net/pipermail/scons-dev/attachments/20160228/45560c42/attachment.html>


More information about the Scons-dev mailing list