[Bug 1258874] Review Request: libmaxminddb - C library for the MaxMind DB file format

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #5 from Jens Lody <fedora@xxxxxxxxxxx> ---
(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
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]