Java scanner don't split JAVACLASSPATH on space

The scanner now splits JAVACLASSPATH before searching only if it is passed
as a scalar (string); it no longer recurses to split elements of a list
value - this is more consistent with other SCons usage.  No longer splits
on space, rather splits on os.pathsep, as considerably more useful for
a search-path variable.  The latter change avoids breaking paths with
an embedded space (usually from Windows).

This is a "breaking change" from the new behavior introduced in 4.4 where
a JAVACLASSPATH like "aaa bbb ccc" would have been split into ["aaa",
"bbb", "ccc"] - now it will be left unchanged in the scanner.  However a
JAVACLASSPATH like "aaa;bbb;ccc" will now be split into ["aaa", "bbb",
"ccc"].  This behavior only affects the scanner - there is no change in
how JAVACLASSPATH gets transformed into the "-classpath ARG" argument
when calling JDK elements.  The former behavior was presumably unintended,
and definitely broke on a space-containing path (e.g. "My Classes".

The unit test is expanded to test for a path string with spaces, and
for a path string containing a search-path separator.

Documentation tweaks: explain better how JAVACLASSPATH (and the other two
Java path variables) can be specified, and whether or not SCons changes
them before calling the JDK commands.  Added note that JAVABOOTCLASSPATH
is no longer useful, and a note about the side effect that SCons always
supplies a "-sourcepath" argument when calling javac.

Fixes #4243

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

0 files changed, 0 insertions, 0 deletions