Switch to per-tool classpath configuration.
Review Request #11651 — Created June 7, 2021 and submitted
The original design for checkstyle and the Java mixin for tools provided
for a single configured classpath, defined asjava_classpath
in the
configuration file or as the$CLASSPATH
environment variable.In practice, this doesn't work too well. Both PMD and checkstyle used
some of the same dependencies, bundled in both their respective jar
files. Depending on the versions, these dependencies would conflict when
both were in a classpath.Rather than having a single classpath, there's now tool-specific
classpaths, defined as tool keys injava_classpaths
in the
configuration file.The
$CLASSPATH
environment variable is no longer supported or
recommended, but if it's set for the process, it will still bubble
through. We may not want to support this long-term, but we get it for
free right now due to howexecute()
works, and it's not worth the
effort of preventing it.
All unit tests passed.
Summary | ID |
---|---|
72596570e7e8b0c93fb24cba6c6b8be9deb90827 |