[Bug 205955] Review Request: gdal - A translator library for raster geospatial data formats

bugzilla at redhat.com bugzilla at redhat.com
Thu Sep 14 15:12:08 UTC 2006


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 at ioa.s.u-tokyo.ac.jp  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.




More information about the Fedora-package-review mailing list