[Bug 1204447] Review Request: python-geoip-geolite2 - GeoIP database access for Python under a BSD license

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

 



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

Julien Enselme <jujens@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |jujens@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |jujens@xxxxxxxxx



--- Comment #1 from Julien Enselme <jujens@xxxxxxxxx> ---
Hi,

Here are some preliminary remarks:

- The github link you provide are for tow packages: python-geoip and
python-geopip-geolite2. From the name of your package, I guess you want to
package the latter. If so, I think you should use pypi to get the tarballs
available at: https://pypi.python.org/pypi/python-geoip-geolite2/2015.0303
- If you want to package both, please create two separate packages
- The changelog is incorrect. The release number is part of the "date line".
- In the %file section, you use %{python2_sitelib}/*. This is too broad, see:
https://fedoraproject.org/wiki/Packaging:Python#Files_to_include

-- 
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]