Mailing-List: contact dev-help@flink.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of ewenstephan@gmail.com
 designates 209.85.223.175 as permitted sender)
MIME-Version: 1.0
Sender: ewenstephan@gmail.com
In-Reply-To: 
 <CAGr9p8Du1usLV3ynq9teWfk8eKaJK+2iOf2a-MkAdC6sFBQ_kg@mail.gmail.com>
References: 
 <CALuGr6ZEdgjoDE3waYV4v5-09wpuWx91tGX1S3AxTiOvpaU+qg@mail.gmail.com>
	<CAGr9p8DBNTE+tW7EH8=WTBc_LXZ-i8YfeEUP2zGvM9_2CB9QwQ@mail.gmail.com>
	<CALuGr6aPi_GNZS8syR02DTMY3D1NWtKjSTiN8WQRvsWmdPRwnA@mail.gmail.com>
	<CAGr9p8Du1usLV3ynq9teWfk8eKaJK+2iOf2a-MkAdC6sFBQ_kg@mail.gmail.com>
Date: Wed, 20 Aug 2014 13:17:56 +0200
Message-ID: 
 <CANC1h_t7J7hYGVtEh5e8k7EDaMjv=8oves241RRQjuYyPyQ0Qw@mail.gmail.com>
Subject: Re: Fail to compile due to too many RAT unapproved licenses
From: Stephan Ewen <sewen@apache.org>
To: dev@flink.incubator.apache.org
Content-Type: multipart/alternative; boundary=089e0111d1be8b1def05010dc12d

--089e0111d1be8b1def05010dc12d
Content-Type: text/plain; charset=UTF-8

Sorry, my fault. Yes, I skipped CI, because no code was changed.

But I forgot to add the new DEPENDENCIES file to the checkstyle ignore list.

Thanks, Robert, for fixing this...


On Wed, Aug 20, 2014 at 9:27 AM, Robert Metzger <rmetzger@apache.org> wrote:

> In theory, yes, but Stephan added the following string into the commit
> message "[ci skip]":
>
> https://github.com/apache/incubator-flink/commit/8bc7894fe5583176a2b35163980fb45657c80bed
> .
> It tells Travis not to start a build.
> We usually do this for non-code changes (like updating some legal files ;)
> ).
>
>
> On Wed, Aug 20, 2014 at 9:22 AM, Henry Saputra <henry.saputra@gmail.com>
> wrote:
>
> > Thanks Robert.
> >
> > Hmm, shouldn't Flink travis CI barf too?
> >
> > - Henry
> >
> > On Tue, Aug 19, 2014 at 11:54 PM, Robert Metzger <rmetzger@apache.org>
> > wrote:
> > > Hi,
> > >
> > > I think rat is being executed when doing an "install" or "verify" (its
> > > bound to the "verify" phase). We have it this way so that our CI finds
> > pull
> > > requests with wrong license headers (happens often to new
> contributors).
> > >
> > > But you are right, the "DEPENDENCIES" file Stephan committed yesterday
> is
> > > marked as unapproved by apache-rat. I'll commit a fix ...
> > >
> > >
> > > Robert
> > >
> > >
> > > On Wed, Aug 20, 2014 at 7:48 AM, Henry Saputra <
> henry.saputra@gmail.com>
> > > wrote:
> > >
> > >> Hi guys,
> > >>
> > >> I just pulled from master and when tried to do mvn install I got
> error:
> > >>
> > >> [ERROR] Failed to execute goal
> > >> org.apache.rat:apache-rat-plugin:0.10:check (default) on project
> > >> flink-parent: Too many files with unapproved license: 1 See RAT report
> > >> in: /home/hsaputra/open/hsaputra/incubator-flink/target/rat.txt ->
> > >> [Help 1]
> > >>
> > >> Are we enabling RAT check on mvn install flow now?
> > >>
> > >> - Henry
> > >>
> >
>

--089e0111d1be8b1def05010dc12d--