I also have overlooked that.
Later, I will always check 'mvn install'. :)
--
Hyunsik Choi
On Thu, Sep 15, 2011 at 12:49 AM, Owen O'Malley wrote:
> On Wed, Sep 14, 2011 at 4:08 AM, Hyunsik Choi wrote:
>> Thank you for reporting.
>
> As a reminder to devs, you should ensure that mvn install succe
Yeah, my bad as well, I actually had that header in my GitHub branch, but
somehow didn't get it into the patch - I did mvn install, I promise! :)
On Wed, Sep 14, 2011 at 8:53 AM, Avery Ching wrote:
> And committers too (my bad) =).
>
>
> On 9/14/11 8:49 AM, Owen O'Malley wrote:
>
>> On Wed, Sep
And committers too (my bad) =).
On 9/14/11 8:49 AM, Owen O'Malley wrote:
On Wed, Sep 14, 2011 at 4:08 AM, Hyunsik Choi wrote:
Thank you for reporting.
As a reminder to devs, you should ensure that mvn install succeeds
before committing the patch. *smile*
-- Owen
On Wed, Sep 14, 2011 at 4:08 AM, Hyunsik Choi wrote:
> Thank you for reporting.
As a reminder to devs, you should ensure that mvn install succeeds
before committing the patch. *smile*
-- Owen
Thank you for reporting.
Yes, this is trial issue, but necessary.
For recording, I just created a jira issue:
https://issues.apache.org/jira/browse/GIRAPH-33
--
Hyunsik Choi
On Wed, Sep 14, 2011 at 7:06 PM, Claudio Martella
wrote:
> I don't know if this issue is worth a JIRA+patch, so I post it
I don't know if this issue is worth a JIRA+patch, so I post it here.
maven install fails as rat is complaining about missing licenses.
Basically GraphState.java is missing Apache license header.
Best,
--
Claudio Martella
claudio.marte...@gmail.com