Home Original page

Fix AddOption handling of spaces by mwichmann · Pull Request #4841 · SCons/scons

@mwichmann 

Fix handling of AddOption for option-arguments with spaces
(when omitting the = sign and/or when an option takes multiple
option-arguments). Arguments unknown at the time the first pass splits
the command line into arguments and targets would put such arguments into
targets, and they remained there even after the AddOption calls were seen.

Signed-off-by: Mats Wichmann <mats@linux.com>
Testcases were added earlier (and recently expanded), but because
they were previously illustarting a shortcoming, there were listed
in the .exclude_tests file, and so not run unless explicitly listed.
Removing that file should let the tests run.

Signed-off-by: Mats Wichmann <mats@linux.com>
The two expected exception message showed SCons paths with enough
detail to include filesystem separators (SCons/Script/Main.py for
example).  Trimmed that back to include only the filename part
so the slash-vs-backslash difference doesn't fail us.

Signed-off-by: Mats Wichmann <mats@linux.com>

@bdbaddog