arturobernalg opened a new pull request, #681:
URL: https://github.com/apache/maven-war-plugin/pull/681

   AbstractWarMojo.buildWebapp() builds its own MavenResourcesExecution for 
webResources filtering but never injected the properties Maven core 
interpolates into the POM itself (maven.build.timestamp, project.baseUri), so 
they were left as literal, unresolved placeholders. maven-resources-plugin 
already recomputes and injects the same two properties for src/main/resources 
filtering (MRESOURCES-99), since that build-time value isn't otherwise 
reachable from a plugin.
   
   Mirror that fix here: a duplicated MavenBuildTimestamp formats the timestamp 
the same way, and buildWebapp() now sets it plus project.baseUri as additional 
properties before filtering, same as ResourcesMojo does.
   
   Fixes #270
   
   ---
   
   To make clear that you license your contribution under
   the [Apache License Version 2.0, January 
2004](https://www.apache.org/licenses/LICENSE-2.0)
   check one of the following:
   
   - [x] I hereby declare this contribution to be licensed under the [Apache 
License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0)
   - [x] I hereby declare that I have filed an [Apache Individual Contributor 
License Agreement](https://www.apache.org/licenses/icla.pdf).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to