Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=910233

Felix Kaechele <fe...@fetzig.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fe...@fetzig.org

--- Comment #4 from Felix Kaechele <fe...@fetzig.org> ---
Some notes:

1. The package's version is 2012.02. Are you sure you want 2012 and not 2013?

2. The package GeoIP
(http://pkgs.fedoraproject.org/cgit/GeoIP.git/tree/GeoIP.spec) owns
%{_datadir}/GeoIP where it also puts a version of GeoIP.dat. The library also
expects other GeoIP databases to be in this path. Maybe you should put your
files there as well instead of %{_datadir}/geoip.
Regarding ownership of files and directories in this case refer to
http://fedoraproject.org/wiki/Packaging:Guidelines#The_directory_is_owned_by_a_package_which_is_not_required_for_your_package_to_function

3. Please refer to
http://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps regarding your
%install section and the creation of the SRPMs (especially downloading the
sources)

-- 
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=qVocKizmEV&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to