https://bugzilla.redhat.com/show_bug.cgi?id=865890
Douglas Schilling Landgraf <dougsl...@redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
--- Comment #5 from Douglas Schilling Landgraf <dougsl...@redhat.com> ---
(In reply to gil cattaneo from comment #3)
> hi
>
> (In reply to Douglas Schilling Landgraf from comment #2)
> > Hi Gil,
> >
> > Please check my comments below:
> >
> > - Maven packages should use new style packaging
> > Note: If possible update your package to latest guidelines
> > See: https://fedoraproject.org/wiki/Packaging:Java#Apache_Maven
> for no use F18 and xmvn binary isnt available
> >
> > - Please do not include license into Source01
> > Source1: http://www.apache.org/licenses/LICENSE-2.0.txt
> >
> > Here suggestions how you can deal with this situation:
> > https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/
> > LicensingGuidelines
> see http://code.google.com/p/concurrentlinkedhashmap/issues/detail?id=38
> >
It's not required to include the license but nice that you have followed the
instructions and notified the project.
Please update your spec removing the Source1 which includes the license and
include (or not) the LICENSE file into it.
Thanks
Douglas
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=EWfPnDXtW1&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
