tbroyer / gwt-maven-plugin

Starting fresh on building GWT projects with Maven
https://tbroyer.github.io/gwt-maven-plugin/
Apache License 2.0
167 stars 39 forks source link

Plugin does not check resource include/exclude when doing conflict-detection with source roots in package-lib #138

Closed foal closed 4 years ago

foal commented 4 years ago

I have a project that copies license from project root to the META-INF folder by maven-resource-plugin. The configuration is the same as described in this question https://stackoverflow.com/questions/28619630/copy-a-resource-file-with-maven-located-at-the-root-of-the-project.

But it caused the following warning

build   10-May-2020 19:00:16    [INFO] --- gwt-maven-plugin:1.0.0:package-lib (default-package-lib) @ org.jresearch.gwt.time ---
build   10-May-2020 19:00:16    [WARNING] Conflicting path between source folder (/var/atlassian/application-data/bamboo/xml-data/build-dir/GWT-GWTTIM-JOB1/src/main/java/, to be added as resource) and resource (/var/atlassian/application-data/bamboo/xml-data/build-dir/GWT-GWTTIM-JOB1/); skipping.
build   10-May-2020 19:00:16    [WARNING] Conflicting path between source folder (/var/atlassian/application-data/bamboo/xml-data/build-dir/GWT-GWTTIM-JOB1/target/generated-sources/annotations/, to be added as resource) and resource (/var/atlassian/application-data/bamboo/xml-data/build-dir/GWT-GWTTIM-JOB1/); skipping.
build   10-May-2020 19:00:16    [INFO] Building jar: /var/atlassian/application-data/bamboo/xml-data/build-dir/GWT-GWTTIM-JOB1/target/org.jresearch.gwt.time-1.4.8-SNAPSHOT.jar

and so no sources in produced JAR.

tbroyer commented 4 years ago

This is by design to keep things simple (in the plugin). The goal is to make simple things easy and more complex things possible, so the plugin bails out on any kind of possible conflict and let's you fix it:

That being said, if you think this can easily be implemented, I'll be happy to review a PR.

foal commented 4 years ago

Thank you! copy-resources and license:update-project-license looks reasonable.