You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by po...@apache.org on 2015/04/14 10:41:33 UTC

svn commit: r1673389 - in /creadur/rat/trunk: RELEASE_NOTES.txt apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java

Author: pottlinger
Date: Tue Apr 14 08:41:32 2015
New Revision: 1673389

URL: http://svn.apache.org/r1673389
Log:
RAT-160: Add build.log to maven default ignores until MINVOKER-190 is solved.


Modified:
    creadur/rat/trunk/RELEASE_NOTES.txt
    creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java
    creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java

Modified: creadur/rat/trunk/RELEASE_NOTES.txt
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/RELEASE_NOTES.txt?rev=1673389&r1=1673388&r2=1673389&view=diff
==============================================================================
--- creadur/rat/trunk/RELEASE_NOTES.txt (original)
+++ creadur/rat/trunk/RELEASE_NOTES.txt Tue Apr 14 08:41:32 2015
@@ -17,6 +17,9 @@ Rat 0.12 (SNAPSHOT)
     * [RAT-188] - Fix problems with AnimalSniffer-plugin when running in GUMP with Java 8.
                   Workaround for https://jira.codehaus.org/browse/MANIMALSNIFFER-45
                   (thanks to Mark Thomas)
+    * [RAT-160] - Ignore build.log that is created by maven-invoker-plugin runs. Temporary fix until invoker plugin improves
+                  configurability of that file. 
+                  (thanks to Christopher Tubbs)
 
    New features
     * [RAT-171] - Maven plugin allows scanning of source code management ignore files. Their contents is added as exclusion/ignored during RAT runs.

Modified: creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java?rev=1673389&r1=1673388&r2=1673389&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java (original)
+++ creadur/rat/trunk/apache-rat-plugin/src/main/java/org/apache/rat/mp/util/ExclusionHelper.java Tue Apr 14 08:41:32 2015
@@ -42,6 +42,7 @@ public final class ExclusionHelper {
                     "cobertura.ser", //
                     "release.properties", //
                     ".repository", // Used by Jenkins when a Maven job uses a private repository that is "Local to the workspace"
+                    "build.log", // RAT-160: until now maven-invoker-plugin runs create a build.log that is not part of a release
                     ".mvn", // Project configuration since Maven 3.3.1 which contains maven.config, jvm.config, extensions.xml
                     "pom.xml.releaseBackup"));
 

Modified: creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java?rev=1673389&r1=1673388&r2=1673389&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java (original)
+++ creadur/rat/trunk/apache-rat-plugin/src/test/java/org/apache/rat/mp/util/ExclusionHelperTest.java Tue Apr 14 08:41:32 2015
@@ -49,7 +49,7 @@ public class ExclusionHelperTest {
                 ECLIPSE_DEFAULT_EXCLUDES.size());
         assertEquals("Did you change the number of idea excludes?", 4,
                 IDEA_DEFAULT_EXCLUDES.size());
-        assertEquals("Did you change the number of mvn excludes?", 6,
+        assertEquals("Did you change the number of mvn excludes?", 7,
                 MAVEN_DEFAULT_EXCLUDES.size());
     }
 
@@ -81,9 +81,9 @@ public class ExclusionHelperTest {
         addMavenDefaults(log, false, exclusion);
         assertTrue(exclusion.isEmpty());
         addMavenDefaults(log, true, exclusion);
-        assertEquals(6, exclusion.size());
+        assertEquals(7, exclusion.size());
         addMavenDefaults(log, true, exclusion);
-        assertEquals(6, exclusion.size());
+        assertEquals(7, exclusion.size());
     }
 
     @Test