https://bugzilla.redhat.com/show_bug.cgi?id=1258874



--- Comment #5 from Jens Lody <[email protected]> ---
(In reply to Jan Včelák from comment #3)
> Thank you for a quick check.
> 
> - I replaced %doc with %license.
> 
> - License field should refer to the content of binary RPM, not source RPM
> [https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License:
> _field]. I've verified that the packages contains only content licensed
> under Apache License 2.0. 
> 
I'm not sure if the mixed source licensing scenario kicks in:
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Mixed_Source_Licensing_Scenario

> - I would like to keep the Group tag as I need the package in EPEL as well.
> 
No problem, it's just a should !
> - Even though rpmlint reports 403, the Source0 works. Checked with wget and
> web browser.
> 
> Updated spec:
> https://jvcelak.fedorapeople.org/review/libmaxminddb/1.1.1-2/libmaxminddb.
> spec
> Updated SRPM:
> https://jvcelak.fedorapeople.org/review/libmaxminddb/1.1.1-2/libmaxminddb-1.
> 1.1-2.fc22.src.rpm

Works here from browser also, maybe a github hickup.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to