From 10c338f3fea963692d8fe0228cde25a64caa0517 Mon Sep 17 00:00:00 2001 From: Konrad Windszus Date: Mon, 9 Mar 2015 10:51:49 +0100 Subject: [PATCH] only propagate git properties to other maven modules --- .../java/pl/project13/maven/git/GitCommitIdMojo.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/main/java/pl/project13/maven/git/GitCommitIdMojo.java b/src/main/java/pl/project13/maven/git/GitCommitIdMojo.java index ddc6e860..983f3aa1 100644 --- a/src/main/java/pl/project13/maven/git/GitCommitIdMojo.java +++ b/src/main/java/pl/project13/maven/git/GitCommitIdMojo.java @@ -99,7 +99,7 @@ public class GitCommitIdMojo extends AbstractMojo { * For details about why you might want to skip this, read this issue: https://github.com/ktoso/maven-git-commit-id-plugin/pull/65 * Basically, injecting into all projects may slow down the build and you don't always need this feature. * - * @parameter default-value="true" + * @parameter default-value="false" */ @SuppressWarnings("UnusedDeclaration") private boolean injectAllReactorProjects; @@ -359,7 +359,7 @@ public void execute() throws MojoExecutionException { } if (injectAllReactorProjects) { - appendPropertiesToReactorProjects(properties); + appendPropertiesToReactorProjects(properties, prefixDot); } } catch (Exception e) { e.printStackTrace(); @@ -408,14 +408,16 @@ private void handlePluginFailure(Exception e) throws MojoExecutionException { } } - private void appendPropertiesToReactorProjects(@NotNull Properties properties) { + private void appendPropertiesToReactorProjects(@NotNull Properties properties, @NotNull String trimmedPrefixWithDot) { for (MavenProject mavenProject : reactorProjects) { Properties mavenProperties = mavenProject.getProperties(); log(mavenProject.getName(), "] project", mavenProject.getName()); for (Object key : properties.keySet()) { - mavenProperties.put(key, properties.get(key)); + if (key.toString().startsWith(trimmedPrefixWithDot)) { + mavenProperties.put(key, properties.get(key)); + } } } }