rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marlon Pierce <mpie...@cs.indiana.edu>
Subject Re: Maven license plugin
Date Thu, 10 Mar 2011 21:49:57 GMT
Hash: SHA1

Rat's command line didn't insert license headers, either, so I set out to debug the source
code in fine Apache fashion.  org.apache.rat.Report.java has two bugs:

* They misspell "license" as "licence" in their options, so "--addLicense" doesn't work, but
"-a" is OK.

* More seriously, the logic of the option parsing is not good: if you don't specify a stylesheet,
you get sent to a broken version of report() that throws away your other command line options.

This raises another question: do we have jira accounts?


On 3/10/11 11:30 AM, Ross Gardler wrote:
> On 10/03/2011 14:32, Marlon Pierce wrote:
>> The links were a helpful start, but is there any way to insert the
>> Apache header in appropriate files with the maven plugin?  That is,
>> I'm looking for the maven pom equivalent to the command line
>> described at http://incubator.apache.org/rat/apache-rat/index.html.
> I know I only ever implemented it as a command line, but don't know if anyone has added
this to the maven plugin.
> However, why do you need it in the plugin? It's only done once right?
> If you really want it to happen in mvn surely mvn can run a java app?
> Ross
>> Thanks--
>> Marlon
>> On 3/9/11 10:46 AM, Ate Douma wrote:
>>> On 09/03/11 16:28, Scott Wilson wrote:
>>>> On 9 Mar 2011, at 14:26, Ross Gardler wrote:
>>>>> I have no opinion on that plugin, I've not used it. I would say
>>>>> be cautious as you have to make sure the headers are only
>>>>> inserted into files that we own copyright on (not files from
>>>>> other projects for example).
>>>>> The ASF has a project that will check for the correct licenses
>>>>> and, optionally, add a license header. I use that. This is a
>>>>> really useful tool, because once setup right it will stop us
>>>>> forgetting to add license headers. However, it has to be set up
>>>>> right as I mention above.
>>>>> See http://incubator.apache.org/rat
>>>> +1 RAT is a really helpful tool
>>> +1
>>> For Apache Portals we have RAT configured in our Maven master pom
>>> in a separate "rat" profile so we can run a check any time we want,
>>> especially before doing a release. See:
>>> https://svn.apache.org/repos/asf/portals/portals-pom/trunk/pom.xml
>>>>> On 09/03/2011 13:27, Marlon Pierce wrote:
>>> Does anyone have an opinion on the maven license plugin
>>> (http://code.google.com/p/maven-license-plugin/)?  I'm trying to
>>> wrap up the gadget container code clean up for contribution and
>>> would like to use it or some other nice tool to insert the Apache
>>> headers (http://www.apache.org/legal/src-headers.html#headers) in
>>> our code without breaking XML, JSP, etc.
>>> Staging area is
>>> http://vigyan.svn.sourceforge.net/viewvc/vigyan/rave-sandbox/, by
>>> the way.
>>> Thanks--
>>> Marlon
>>>>> -- rgardler@apache.org @rgardler
>> -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.16
>> (Darwin) Comment: Using GnuPG with Mozilla -
>> http://enigmail.mozdev.org/
>> Tp5vNG5BbCplRo3o6BWBO99mM7JPg77Mat2AevoE79jpk7Tk9BdhS06gZkGWrHaP
>> HFoev3cz9Is94RngzNdSjtimwj7aKFZL8T6+/M9+1+3JVGr9obR9l/kIUd+k8KKX
>> u1IgUzhSRjV+vM2mY9n/4ZzUyAsSS2jRAXZR3cj7EeoAehX4gGywX6wLTYRIyQEm
>> j3oAEPqZ5bI4UX6dLyaf8zjTADg97Py4tAHEafmmLUzazQ4AV3du+6DAwAbyJo5s
>> q8Xe+q4b24CtjWGJaARqFvu/owqHckHn4bDCn4H1Rkw06l+YbOh600O4Y3CFTxw=
>> =TONs -----END PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.16 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/


View raw message