Skip to content

Commit 90960d1

Browse files
committed
Code review: improve DefaultCompilerManager.ERROR_MESSAGE
Add more detail concerning possible user errors like misspelling the compiler ID or missing dependencies for a compiler. Relates to #347.
1 parent ad1164b commit 90960d1

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

plexus-compiler-manager/src/main/java/org/codehaus/plexus/compiler/manager/DefaultCompilerManager.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@
3939
@Named
4040
public class DefaultCompilerManager implements CompilerManager {
4141
private static final String ERROR_MESSAGE = "Compiler '{}' could not be instantiated or injected properly. "
42-
+ "Running the build with -Dsisu.debug, looking for exceptions might help.";
42+
+ "If you spelled the compiler ID correctly and all necessary dependencies are on the classpath, "
43+
+ "then next you can try running the build with -Dsisu.debug, looking for exceptions.";
4344
private static final String ERROR_MESSAGE_DETAIL = "TypeNotPresentException caused by UnsupportedClassVersionError "
4445
+ "might indicate, that the compiler needs a more recent Java runtime. "
4546
+ "IllegalArgumentException in ClassReader.<init> might mean, that you need to upgrade Maven.";

0 commit comments

Comments
 (0)