Skip to content

Commit

Permalink
Handle missing repository policy values
Browse files Browse the repository at this point in the history
See https://maven.apache.org/pom.html#Repositories for details

Change-Id: Ic7f8c438991fd41949d27ae28dd7f74bbe6d0d20
  • Loading branch information
talios committed Oct 17, 2023
1 parent 4ac6409 commit 6ce2153
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 5 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@

All notable changes to this project will be documented in this file.

== 2.38

- Fix NPE in Repository Policy retreival
- Fix distribution management snapshot repository load (Ricardo Caldeira <[email protected]>)

== 2.37

- Drop maven-compat and update dependendies
Expand Down
4 changes: 2 additions & 2 deletions README.adoc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
= Tiles Maven Plugin - Version 2.37
= Tiles Maven Plugin - Version 2.38

image:https://travis-ci.org/repaint-io/maven-tiles.svg[caption="Travis Build Status"] image:https://badges.gitter.im/repaint-io/maven-tiles.svg[link="https://gitter.im/repaint-io/maven-tiles?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge"]

Expand Down Expand Up @@ -63,7 +63,7 @@ you specify a configuration.
<plugin>
<groupId>io.repaint.maven</groupId>
<artifactId>tiles-maven-plugin</artifactId>
<version>2.37</version>
<version>2.38</version>
<extensions>true</extensions>
<configuration>
<filtering>false</filtering>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,8 @@ import static io.repaint.maven.tiles.GavUtil.artifactName
import static io.repaint.maven.tiles.GavUtil.modelGav
import static io.repaint.maven.tiles.GavUtil.modelRealGa
import static io.repaint.maven.tiles.GavUtil.parentGav

import static org.apache.maven.artifact.repository.ArtifactRepositoryPolicy.CHECKSUM_POLICY_WARN
import static org.apache.maven.artifact.repository.ArtifactRepositoryPolicy.UPDATE_POLICY_ALWAYS
/**
* Fetches all dependencies defined in the POM `configuration`.
*
Expand Down Expand Up @@ -343,8 +344,12 @@ class TilesMavenLifecycleParticipant extends AbstractMavenLifecycleParticipant {
}

ArtifactRepositoryPolicy getArtifactRepositoryPolicy(RepositoryPolicy policy) {
return new ArtifactRepositoryPolicy(Boolean.valueOf(policy.enabled),
policy.updatePolicy, policy.checksumPolicy)
if (policy != null) {
return new ArtifactRepositoryPolicy(policy.isEnabled(), policy.updatePolicy, policy.checksumPolicy)
} else {
return new ArtifactRepositoryPolicy(true, UPDATE_POLICY_ALWAYS, CHECKSUM_POLICY_WARN)
}

}

/**
Expand Down

0 comments on commit 6ce2153

Please sign in to comment.