You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Gary Gregory <ga...@gmail.com> on 2019/03/09 22:44:54 UTC

[Parent] RAT check

Hi all:

How about making apache-rat:check run automatically in commons-parent? In
the Maven validate phase?

That would mean one less thing to check when integrating a patch and
validating an RC.

Gary

Re: [Parent] RAT check

Posted by Eitan Adler <li...@eitanadler.com>.
(please make sure to CC me on replies)

+1

On Sat, 9 Mar 2019 at 19:51, Bruno P. Kinoshita <ki...@apache.org> wrote:
>
>
>
> Also cannot think of a reason why not. So +1
>
>
>
>
> On Sunday, 10 March 2019, 2:06:05 pm NZDT, Rob Tompkins <ch...@gmail.com> wrote:
>
>
>
>
>
>
>
> > On Mar 9, 2019, at 7:40 PM, Gilles Sadowski <gi...@gmail.com> wrote:
> >
> >> Le sam. 9 mars 2019 à 23:45, Gary Gregory <ga...@gmail.com> a écrit :
> >>
> >> Hi all:
> >>
> >> How about making apache-rat:check run automatically in commons-parent? In
> >> the Maven validate phase?
> >>
> >> That would mean one less thing to check when integrating a patch and
> >> validating an RC.
> >
> > +1
> > [Why not?]
> >
>
> Agree +1
>
>
> > Gilles
> >
> >
> >>
> >> Gary
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>


-- 
Eitan Adler

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by "Bruno P. Kinoshita" <ki...@apache.org>.

Also cannot think of a reason why not. So +1




On Sunday, 10 March 2019, 2:06:05 pm NZDT, Rob Tompkins <ch...@gmail.com> wrote: 







> On Mar 9, 2019, at 7:40 PM, Gilles Sadowski <gi...@gmail.com> wrote:
> 
>> Le sam. 9 mars 2019 à 23:45, Gary Gregory <ga...@gmail.com> a écrit :
>> 
>> Hi all:
>> 
>> How about making apache-rat:check run automatically in commons-parent? In
>> the Maven validate phase?
>> 
>> That would mean one less thing to check when integrating a patch and
>> validating an RC.
> 
> +1
> [Why not?]
> 

Agree +1


> Gilles
> 
> 
>> 
>> Gary
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org

> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Rob Tompkins <ch...@gmail.com>.

> On Mar 9, 2019, at 7:40 PM, Gilles Sadowski <gi...@gmail.com> wrote:
> 
>> Le sam. 9 mars 2019 à 23:45, Gary Gregory <ga...@gmail.com> a écrit :
>> 
>> Hi all:
>> 
>> How about making apache-rat:check run automatically in commons-parent? In
>> the Maven validate phase?
>> 
>> That would mean one less thing to check when integrating a patch and
>> validating an RC.
> 
> +1
> [Why not?]
> 

Agree +1


> Gilles
> 
> 
>> 
>> Gary
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Gilles Sadowski <gi...@gmail.com>.
Le sam. 9 mars 2019 à 23:45, Gary Gregory <ga...@gmail.com> a écrit :
>
> Hi all:
>
> How about making apache-rat:check run automatically in commons-parent? In
> the Maven validate phase?
>
> That would mean one less thing to check when integrating a patch and
> validating an RC.

+1
[Why not?]

Gilles


>
> Gary

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Gary Gregory <ga...@gmail.com>.
Running in verify is OK as well.

Gary

On Sun, Mar 10, 2019, 06:35 sebb <sebbaz@gmail.com wrote:

> OK, but please provide a property to switch it off on the command-line.
>
> I think it is going to be very annoying for local development,
> especially if it runs at such an early phase.
>
> It could also be done in a much later phase, e.g. verify
> That would certainly be sufficient for RC creation.
>
> On Sun, 10 Mar 2019 at 08:43, Pascal Schumacher
> <pa...@gmx.net> wrote:
> >
> > +1
> >
> > Am 09.03.2019 um 23:44 schrieb Gary Gregory:
> > > Hi all:
> > >
> > > How about making apache-rat:check run automatically in commons-parent?
> In
> > > the Maven validate phase?
> > >
> > > That would mean one less thing to check when integrating a patch and
> > > validating an RC.
> > >
> > > Gary
> > >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [Parent] RAT check

Posted by sebb <se...@gmail.com>.
OK, but please provide a property to switch it off on the command-line.

I think it is going to be very annoying for local development,
especially if it runs at such an early phase.

It could also be done in a much later phase, e.g. verify
That would certainly be sufficient for RC creation.

On Sun, 10 Mar 2019 at 08:43, Pascal Schumacher
<pa...@gmx.net> wrote:
>
> +1
>
> Am 09.03.2019 um 23:44 schrieb Gary Gregory:
> > Hi all:
> >
> > How about making apache-rat:check run automatically in commons-parent? In
> > the Maven validate phase?
> >
> > That would mean one less thing to check when integrating a patch and
> > validating an RC.
> >
> > Gary
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Pascal Schumacher <pa...@gmx.net>.
+1

Am 09.03.2019 um 23:44 schrieb Gary Gregory:
> Hi all:
>
> How about making apache-rat:check run automatically in commons-parent? In
> the Maven validate phase?
>
> That would mean one less thing to check when integrating a patch and
> validating an RC.
>
> Gary
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Benedikt Ritter <br...@apache.org>.
+1 to anything that makes releasing easier.

Am Sa., 9. März 2019 um 23:45 Uhr schrieb Gary Gregory <
garydgregory@gmail.com>:

> Hi all:
>
> How about making apache-rat:check run automatically in commons-parent? In
> the Maven validate phase?
>
> That would mean one less thing to check when integrating a patch and
> validating an RC.
>
> Gary
>

Re: [Parent] RAT check

Posted by Gary Gregory <ga...@gmail.com>.
On Mon, Mar 11, 2019 at 6:58 AM Jochen Wiedmann <jo...@gmail.com>
wrote:

> On Sat, Mar 9, 2019 at 11:45 PM Gary Gregory <ga...@gmail.com>
> wrote:
>
> > How about making apache-rat:check run automatically in commons-parent? In
> > the Maven validate phase?
>
> Not being a Maven expert: When would that check run, exactly? If that
> would include a simple "mvn clean install", then I'd definitely be -1,
> A build should be quick, and painless. That could not exclude a rat
> check.
>
> Btw: For that very reason, I have written a Rat alternative [1], which
> is much faster than Rat. My intention was to have this running inside
> a Git hook. However, if I should find someone, who's getting me
> started with Eclipse plugin programming, then I might add such a
> plugin.
>

RAT performance has never been a issue for me. Do you have a project with
100,000 source files or something like that?

Gary

>
> Jochen
>
> 1: https://github.com/jochenw/qse
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [Parent] RAT check

Posted by Otto Fowler <ot...@gmail.com>.
Maybe put it in a profile, and you can document that that profile will be
used to check _and_ run that profile for the travis builds, such that a
failed travis build will show that the pr is invalid.


On March 12, 2019 at 09:39:44, Gary Gregory (garydgregory@gmail.com) wrote:

On Mon, Mar 11, 2019 at 4:59 PM Jochen Wiedmann <jo...@gmail.com>
wrote:

> On Mon, Mar 11, 2019 at 5:58 PM Gary Gregory <ga...@gmail.com>
> wrote:
>
> > The idea, for me, is that a contributor should not be able to test and
> > submit a PR with a failing RAT check.
>
> If so, then I'd suggest a server-side Git hook for that.
>

I do not like this idea because I want to know BEFORE I commit or propose a
PR that something is wrong.

Gary

>
> Jochen
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [Parent] RAT check

Posted by Gary Gregory <ga...@gmail.com>.
On Mon, Mar 11, 2019 at 4:59 PM Jochen Wiedmann <jo...@gmail.com>
wrote:

> On Mon, Mar 11, 2019 at 5:58 PM Gary Gregory <ga...@gmail.com>
> wrote:
>
> > The idea, for me, is that a contributor should not be able to test and
> > submit a PR with a failing RAT check.
>
> If so, then I'd suggest a server-side Git hook for that.
>

I do not like this idea because I want to know BEFORE I commit or propose a
PR that something is wrong.

Gary

>
> Jochen
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [Parent] RAT check

Posted by Jochen Wiedmann <jo...@gmail.com>.
On Mon, Mar 11, 2019 at 5:58 PM Gary Gregory <ga...@gmail.com> wrote:

> The idea, for me, is that a contributor should not be able to test and
> submit a PR with a failing RAT check.

If so, then I'd suggest a server-side Git hook for that.

Jochen

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org

Re: [Parent] RAT check

Posted by Gary Gregory <ga...@gmail.com>.
On Mon, Mar 11, 2019 at 6:58 AM Jochen Wiedmann <jo...@gmail.com>
wrote:

> On Sat, Mar 9, 2019 at 11:45 PM Gary Gregory <ga...@gmail.com>
> wrote:
>
> > How about making apache-rat:check run automatically in commons-parent? In
> > the Maven validate phase?
>
> Not being a Maven expert: When would that check run, exactly? If that
> would include a simple "mvn clean install", then I'd definitely be -1,
> A build should be quick, and painless. That could not exclude a rat
> check.
>

The idea, for me, is that a contributor should not be able to test and
submit a PR with a failing RAT check.

So assuming that a contributor does 'mvn clean package' to test, then the
build should fail.

This also would let the RM building a RC notice that the build fails.

Gary

>
> Btw: For that very reason, I have written a Rat alternative [1], which
> is much faster than Rat. My intention was to have this running inside
> a Git hook. However, if I should find someone, who's getting me
> started with Eclipse plugin programming, then I might add such a
> plugin.
>
> Jochen
>
> 1: https://github.com/jochenw/qse
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Re: [Parent] RAT check

Posted by Jochen Wiedmann <jo...@gmail.com>.
On Sat, Mar 9, 2019 at 11:45 PM Gary Gregory <ga...@gmail.com> wrote:

> How about making apache-rat:check run automatically in commons-parent? In
> the Maven validate phase?

Not being a Maven expert: When would that check run, exactly? If that
would include a simple "mvn clean install", then I'd definitely be -1,
A build should be quick, and painless. That could not exclude a rat
check.

Btw: For that very reason, I have written a Rat alternative [1], which
is much faster than Rat. My intention was to have this running inside
a Git hook. However, if I should find someone, who's getting me
started with Eclipse plugin programming, then I might add such a
plugin.

Jochen

1: https://github.com/jochenw/qse

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org