Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gdal - A translator library for raster geospatial data formats https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=205955 ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2006-09-14 11:11 EST ------- Third review. 1. From http://fedoraproject.org/wiki/Packaging/Guidelines : * Legal - Legal issue needs checking (as argued above). * Code Vs Content - Well, again, what are the following files? /usr/share/gdal/seed_2d.dgn (in gdal) /usr/share/gdal/seed_3d.dgn (in gdal) I don't know what these files are. If you also don't know what these are, remove these files. 2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines : = Nothing. 3. Other things I have noticed: * Well, when including man files in rpm package, the format like "gdal_contour.1*" (not gdal_contour.1.gz) is prefered. * Also, when includes all .py, .pyo, .pyc files, the format like "%{python_sitearch}/gdal.py*" (not writing 3 files seperately) is prefered. * Check again what you want to support. * Perhaps it must be considered why HDF4 support is disabled. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review