]
Jan-Willem Gmelig Meyling commented on SHRINKWRAP-521:
------------------------------------------------------
is merged and released with
Plexus Compiler 2.8.5, so updating that dependency should resolve the issue. Nevertheless,
it may still be useful to wire up a logger so that output by the Plexus Compiler can be
propagated.
Can't compile without compile without JAVA_HOME set, due to NPE
in JavacCompiler
--------------------------------------------------------------------------------
Key: SHRINKWRAP-521
URL:
https://issues.jboss.org/browse/SHRINKWRAP-521
Project: ShrinkWrap
Issue Type: Bug
Reporter: Jan-Willem Gmelig Meyling
Priority: Trivial
If JAVA_HOME is not set, the Plexus JavacCompiler will fallback to "which
javac". However, this statement is logged without a proper check that checks whether
logging is enabled (see my pull request
https://github.com/codehaus-plexus/plexus-compiler/pull/52 ). Because Shrinkwrap does not
attach a logger to the JavacCompiler, and this issue in Plexus Compiler, the JavacCompiler
will throw a null pointer exception during compilation causing the test run to fail.
I've already proposed a PR to Plexus Compiler, but perhaps it is a good idea to
attach a logger anyway so that compilation output can be printed to the logging output,
which will also resolve this issue.