Re: [VOTE] Release Commons Numbers 1.3 based on RC1

On Thu, 16 Apr 2026 at 15:33, Gilles Sadowski <[email protected]> wrote:

>
> >
> > RAT Report:
> >
> >
> https://dist.apache.org/repos/dist/dev/commons/numbers/1.3-RC1/site/rat-report.html
>
> This contains:
> ---CUT---
> *****************************************************
> Files with unapproved licenses
> *****************************************************
>
>   /src/site/resources/release-notes/RELEASE-NOTES-1.0-beta1.txt
>   /src/site/resources/release-notes/RELEASE-NOTES-1.0.txt
>   /src/site/resources/release-notes/RELEASE-NOTES-1.1.txt
>   /src/site/resources/release-notes/RELEASE-NOTES-1.2.txt
>   /src/site/resources/release-notes/RELEASE-NOTES-1.3.txt
> ---CUT---
>

This is weird. The pom contains under build and reporting:

      <plugin>
        <groupId>org.apache.rat</groupId>
        <artifactId>apache-rat-plugin</artifactId>
        <version>${commons.rat.version}</version>
        <configuration>
         <!--  Should agree with apache-rat-plugin config under <build> -->
          <inputExcludes>
            <exclude>src/test/resources/data/**</exclude>

<exclude>src/site/resources/release-notes/RELEASE-NOTES-*.txt</exclude>
            <exclude>dist-archive/**</exclude>
            <exclude>**/site-content/**</exclude>
          </inputExcludes>
        </configuration>
      </plugin>

Note that from apache-rat:0.17 the <exclude> tag is deprecated. I updated
to the newer <inputExcludes>. This seems to allow these files to pass the
apache-rat:check goal. Looking at commons parent it uses <inputExclude> as
the tag here in the build configuration but still has the old <excludes>
tag in the reporting section.

The apache-rat documentation is not clear what tag to use to replace the
<excludes> tag. I thought I had it correct as the build did not fail under
rat:check. I will try and reproduce this and then correct the rat
configuration.