jlengrand/quarkus
1
0
Fork 0
mirror of https://github.com/jlengrand/quarkus.git synced 2026-03-10 08:41:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove test-related entries in maven-compiler-plugin

Browse Source 
Some properties (like `testSource` and `testTarget`) are valid only for the `testCompile` mojo, however they are allowed to be declared as part of the maven-compiler-plugin configuration

Co-authored-by: Justin Lee <julee@redhat.com>
This commit is contained in:
George Gastaldi
2020-02-14 12:47:07 -03:00
parent 3c27c5c385
commit ec5bccf43c
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
devtools/maven/src/main/java/io/quarkus/maven/DevMojo.java
View File

@@ -370,9 +370,15 @@ public class DevMojo extends AbstractMojo {
}
private void executeCompileGoal(Plugin plugin, String groupId, String artifactId) throws MojoExecutionException {
Xpp3Dom configuration = (Xpp3Dom) plugin.getConfiguration();
if (configuration == null) {
configuration = MojoExecutor.configuration();
Xpp3Dom configuration = MojoExecutor.configuration();
Xpp3Dom pluginConfiguration = (Xpp3Dom) plugin.getConfiguration();
if (pluginConfiguration != null) {
//Filter out `test*` configurations
for (Xpp3Dom child : pluginConfiguration.getChildren()) {
if (!child.getName().startsWith("test")) {
configuration.addChild(child);
}
}
}
MojoExecutor.executeMojo(
MojoExecutor.plugin(