You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by gn...@apache.org on 2022/10/05 14:45:30 UTC
[maven] branch maven-3.9.x updated: Make sure RAT plugin ignores files in the target folder (#813) (#815)
This is an automated email from the ASF dual-hosted git repository.
gnodet pushed a commit to branch maven-3.9.x
in repository https://gitbox.apache.org/repos/asf/maven.git
The following commit(s) were added to refs/heads/maven-3.9.x by this push:
new a7a5b8169 Make sure RAT plugin ignores files in the target folder (#813) (#815)
a7a5b8169 is described below
commit a7a5b8169ba76a8169898735c91856b3722af639
Author: Guillaume Nodet <gn...@gmail.com>
AuthorDate: Wed Oct 5 16:44:10 2022 +0200
Make sure RAT plugin ignores files in the target folder (#813) (#815)
* Make sure RAT plugin ignores files in the target folder
* Fix gitignore so that it can work with rat, do not inherit execution in children, use a single place to configure rat
* Exclude .asf.yaml from rat check
* Add repo/ to the .gitignore file as it's often used for local repository
---
.gitignore | 3 ++-
pom.xml | 48 ++++++++++++++++++++++--------------------------
2 files changed, 24 insertions(+), 27 deletions(-)
diff --git a/.gitignore b/.gitignore
index e7c899857..30f07e74b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,4 +1,4 @@
-target/
+**/target/**
.project
.classpath
.settings/
@@ -14,3 +14,4 @@ target/
.factorypath
.checkstyle
.vscode/
+repo/
diff --git a/pom.xml b/pom.xml
index 53c9fd51a..ad3df8cbf 100644
--- a/pom.xml
+++ b/pom.xml
@@ -583,25 +583,6 @@ under the License.
<artifactId>buildnumber-maven-plugin</artifactId>
<version>1.4</version>
</plugin>
- <plugin>
- <groupId>org.apache.rat</groupId>
- <artifactId>apache-rat-plugin</artifactId>
- <configuration>
- <excludes>
- <exclude>**/.gitattributes</exclude>
- <exclude>src/test/resources*/**</exclude>
- <exclude>src/test/projects/**</exclude>
- <exclude>src/test/remote-repo/**</exclude>
- <exclude>**/*.odg</exclude>
- <!--
- ! Excluded the license files itself cause they do not have have a license of themselves.
- -->
- <exclude>src/main/appended-resources/licenses/EPL-1.0.txt</exclude>
- <exclude>src/main/appended-resources/licenses/unrecognized-aopalliance-1.0.txt</exclude>
- <exclude>src/main/appended-resources/licenses/unrecognized-javax.annotation-api-1.3.2.txt</exclude>
- </excludes>
- </configuration>
- </plugin>
<!--This plugin's configuration is used to store Eclipse m2e settings only. It has no influence on the Maven build itself.-->
<plugin>
<groupId>org.eclipse.m2e</groupId>
@@ -668,13 +649,28 @@ under the License.
<plugin>
<groupId>org.apache.rat</groupId>
<artifactId>apache-rat-plugin</artifactId>
- <configuration>
- <excludes combine.children="append">
- <exclude>bootstrap/**</exclude>
- <exclude>README.bootstrap.txt</exclude>
- <exclude>README.md</exclude>
- </excludes>
- </configuration>
+ <executions>
+ <execution>
+ <id>rat-check</id>
+ <inherited>false</inherited>
+ <configuration>
+ <excludes>
+ <exclude>**/.gitattributes</exclude>
+ <exclude>src/test/resources*/**</exclude>
+ <exclude>src/test/projects/**</exclude>
+ <exclude>src/test/remote-repo/**</exclude>
+ <exclude>**/*.odg</exclude>
+ <exclude>.asf.yaml</exclude>
+ <!--
+ ! Excluded the license files itself cause they do not have have a license of themselves.
+ -->
+ <exclude>src/main/appended-resources/licenses/EPL-1.0.txt</exclude>
+ <exclude>src/main/appended-resources/licenses/unrecognized-aopalliance-1.0.txt</exclude>
+ <exclude>src/main/appended-resources/licenses/unrecognized-javax.annotation-api-1.3.2.txt</exclude>
+ </excludes>
+ </configuration>
+ </execution>
+ </executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>