Interesting. As part of the release I always run
mvn -P rat -DskipTests clean verify
And it failed to complain about that.
As far as the MD5’s I seem to recall there was a recommendation not to use them
but it wasn’t a mandate. I’m not sure where that email is.
Ralph
> On Jul 23, 2018, at
On Mon, Jul 23, 2018 at 10:59 AM Matt Sicker wrote:
> Can't we just set up our rat rules globally rather than via a profile? It'd
> be nice if CI could catch missing license headers before working on making
> an RC.
>
+1. I do not see why we need a profile.
Gary
>
> On Mon, 23 Jul 2018 at
On Mon, Jul 23, 2018 at 9:09 AM Ralph Goers
wrote:
> Interesting. As part of the release I always run
>
> mvn -P rat -DskipTests clean verify
>
> And it failed to complain about that.
>
I first ran 'mvn clean install' which I should not have. But since I did
that first and THEN ran the RAT
Can't we just set up our rat rules globally rather than via a profile? It'd
be nice if CI could catch missing license headers before working on making
an RC.
On Mon, 23 Jul 2018 at 11:30, Ralph Goers
wrote:
> When you do not use -P rat you are not using our configuration so you are
> including
On Mon, Jul 23, 2018 at 9:09 AM Ralph Goers
wrote:
> Interesting. As part of the release I always run
>
> mvn -P rat -DskipTests clean verify
>
> And it failed to complain about that.
>
> As far as the MD5’s I seem to recall there was a recommendation not to use
> them but it wasn’t a mandate.
When you do not use -P rat you are not using our configuration so you are
including files we specifically exclude. It is bound to fail if you don’t use
the profile.
Ralph
> On Jul 23, 2018, at 8:15 AM, Gary Gregory wrote:
>
> On Mon, Jul 23, 2018 at 9:09 AM Ralph Goers
> wrote:
>
>>
Per Apache policy, we should NOT distribute MD5 hash files. This is not a
blocker as long as these files do not make it past this RC out to the dirt
release server.
Gary
On Sun, Jul 22, 2018 at 10:44 PM Ralph Goers
wrote:
> This is a vote to release Log4j 2.11.1, the next version of the Log4j
+1
>From src zip: ASC OK, SHA1 OK.
Maven RAT check OK using 'mvn apache-rat:check'.
Clirr check using 'mvn clirr:check -pl log4j-api,log4j-1.2-api'
Building 'mvn clean install' OK using:
Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe;
2018-06-17T12:33:14-06:00)
Maven home:
Review still in progress, please ignore below.
On Mon, Jul 23, 2018 at 8:54 AM Gary Gregory wrote:
> +1
>
> From src zip: ASC OK, SHA1 OK.
>
> Maven RAT check OK using 'mvn apache-rat:check'.
>
> Clirr check using 'mvn clirr:check -pl log4j-api,log4j-1.2-api'
>
> Building 'mvn clean install' OK
+1
- From src zip: ASC OK, SHA1 OK.
- Maven RAT check FAILs using 'mvn apache-rat:check':
Files with unapproved licenses:
C:/temp/rc/log4j/apache-log4j-2.11.1-src/log4j-cassandra/.toDelete
Not a blocker since this is not a "source" file but should be fixed ASAP
IMO to make RC reviews
10 matches
Mail list logo