activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Clebert Suconic <clebert.suco...@gmail.com>
Subject Re: [DISCUSSION] License Headers on Artemis/package-info.java
Date Wed, 03 Jun 2015 02:41:11 GMT
Nice one :)

On Tue, Jun 2, 2015 at 10:28 PM, Thiago Kronig <thiagokronig@gmail.com> wrote:
> https://github.com/apache/activemq-artemis/pull/11
>
> On Tue, Jun 2, 2015 at 5:28 PM Clebert Suconic <clebert.suconic@gmail.com>
> wrote:
>
>> On Tue, Jun 2, 2015 at 12:47 PM, Ganesh Murthy <gmurthy@redhat.com> wrote:
>> > I noticed that many java files in ActiveMQ have the license header as a
>> javadoc but the HTML page does not show the license header. Example -
>>
>> I agree, but even though the issue is only on package-info... I think
>> these should be updated to /* on all the license headers. (any
>> volunteers to take the task? : )
>>
>> We copied these license headers verbatin from ActiveMQ-5, but IMO it
>> would make more sense to be regular comments instead of javadocs.
>>
>>
>> Any objections?
>>
>> Anyone volunteering to send a PR? :)
>>



-- 
Clebert Suconic
http://community.jboss.org/people/clebert.suconic@jboss.com
http://clebertsuconic.blogspot.com

Mime
View raw message