Skip to content

Commit fcd414d

Browse files
author
TheSnoozer
committed
Merge remote-tracking branch 'origin/master' into 674
2 parents 953b94e + 4c638aa commit fcd414d

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -556,7 +556,7 @@
556556
<plugin>
557557
<groupId>org.codehaus.mojo</groupId>
558558
<artifactId>exec-maven-plugin</artifactId>
559-
<version>3.1.1</version>
559+
<version>3.2.0</version>
560560
<inherited>false</inherited>
561561
<executions>
562562
<execution>

src/main/java/pl/project13/maven/git/GitCommitIdMojo.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1420,17 +1420,17 @@ public boolean shouldPropertiesEscapeUnicode() {
14201420
}
14211421

14221422
private void publishToAllSystemEnvironments(
1423-
LogInterface log,
1424-
Properties propertiesToPublish,
1425-
Properties contextProperties) {
1423+
@Nonnull LogInterface log,
1424+
@Nonnull Properties propertiesToPublish,
1425+
@Nullable Properties contextProperties) {
14261426
publishPropertiesInto(propertiesToPublish, project.getProperties());
14271427
// some plugins rely on the user properties (e.g. flatten-maven-plugin)
14281428
publishPropertiesInto(propertiesToPublish, session.getUserProperties());
14291429

14301430
if (injectAllReactorProjects) {
14311431
Properties diffPropertiesToPublish = new Properties();
14321432
propertiesToPublish.forEach((k, v) -> {
1433-
if (!contextProperties.contains(k)) {
1433+
if (contextProperties == null || !contextProperties.contains(k)) {
14341434
diffPropertiesToPublish.setProperty(k.toString(), v.toString());
14351435
}
14361436
});

0 commit comments

Comments
 (0)