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

Julien Enselme <juj...@jujens.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |juj...@jujens.eu
           Assignee|nob...@fedoraproject.org    |juj...@jujens.eu



--- Comment #1 from Julien Enselme <juj...@jujens.eu> ---
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to