Red Hat Bugzilla – Bug 527706
Review Request: mingw32-libgeotiff - MinGW port of libgeotiff Georeferenced image library
Last modified: 2009-12-11 13:22:51 EST
Spec URL: http://www.adsllc.com/tmp/mingw32-libgeotiff.spec SRPM URL: http://www.adsllc.com/tmp/mingw32-libgeotiff-1.3.0-svn1664.fc11.src.rpm Description: MinGW port of libgeotiff Georeferenced image library GeoTIFF represents an effort by over 160 different remote sensing, GIS, cartographic, and surveying related companies and organizations to establish a TIFF based interchange format for georeferenced raster imagery. This is my second package submission, and I am looking for a sponsor. See also: mingw32-proj. The source for this is SVN rev 1664 from https://svn.osgeo.org/metacrs/geotiff/trunk/libgeotiff, then './autogen.sh' and 'mingw32-configure' and 'make dist'. The latest release has build system issues that make porting difficult so a SVN snapshot is used instead for now - I expect libgeotiff-1.3.0 to fix this when it is released.
Fix a number of self-caught packing problems. rpmlint mingw32-libgeotiff.spec mingw32-libgeotiff-1.3.0-1.svn1664.fc11.src.rpm mingw32-libgeotiff-1.3.0-1.svn1664.fc11.noarch.rpm 2 packages and 1 specfiles checked; 0 errors, 0 warnings. SPEC: http://adsllc.fedorapeople.org/rpmbuild/SPECS/mingw32-libgeotiff.spec SRPM (won't build in Koji yet due to mingw32-proj dependency): http://adsllc.fedorapeople.org/rpmbuild/SRPMS/mingw32-libgeotiff-1.3.0-1.svn1664.fc11.src.rpm
I'll review it.
Are there any reasons to use pre-1.3.0 version instead of stable 1.2.5? We are still providing 1.2.5 in main Fedora. Other things looks sane.
And, please, add more files to %doc - LICENSE, at least (actually, you must add it).
#3 - 1.2.5 has a broken build system that doesn't work with mingw32-configure. The 1.3.0 release should include the fix, but svn 1664 works for now. #4 - This command svn co -r 1664 https://svn.osgeo.org/metacrs/geotiff/trunk/libgeotiff ; cd libgeotiff ; ./autogen.sh ; mingw32-configure ; make dist Does not include the LICENSE file in resulting libgeotiff-1.3.0.tar.gz. The review guidelines say this must be packaged "if and only if" the source package includes it, which is does not. I'll look into sending a fix upstream since it probably should be included in later releases.
Upstream packaging bug submitted so that LICENSE (and other things) get included in the tarball: http://trac.osgeo.org/gdal/ticket/3192
Ping. I believe the package meets the requirements as-is. Comment #3 has been addressed (I'm using SVN instead of a release for a good reason). Comment #4 has also been addressed (LICENSE is not in the upstream source tarball, so I cannot include it).
Dave, I didn't made the full review only because mingw32-proj is still unavailable in Koji, and, therefore, I can't make test rebuild (which is necessary for me, for the review process). I realized that you just requested a movement them to stable, so I'll do a review in a couple of days or so.
Got it - thanks. Here is the mingw32-proj Koji page: http://koji.fedoraproject.org/koji/packageinfo?packageID=9367
Koji scratchbuild for F-11 (success): http://koji.fedoraproject.org/koji/taskinfo?taskID=1789938 Koji scratchbuild for F-12 (success): http://koji.fedoraproject.org/koji/taskinfo?taskID=1789940 Koji scratchbuild for EL-5 (FAILURE!): http://koji.fedoraproject.org/koji/taskinfo?taskID=1789943 Review: + rpmlint is silent +/- The package is named according to the Package Naming Guidelines, except one small issue in the versioning scheme - since you're packaging pre-release (from VCS), then "Release" field should start from 0. E.g. Release: 0.1.svn1664%{?dist} + The spec file name matches the base package %{name}, in the format %{name}.spec. + The package meets the Packaging Guidelines . + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The License field in the package spec file matches the actual license. +/- The file, containing the text of the license(s) for the package, should be included in %doc, but, unfortunately, upstream does not ship this file in their VCS. This issue was reported upstream (see comments above) and may be ignoired here. + The spec file is written in American English. + The spec file for the package is legible. +/- The sources used to build the package, must match the upstream source, as provided in the spec URL. However, since this is a pre-release and no tarball was provided by upstream, the only way to check the integrity of the sources, is to make diff against freshly checked out VCS tree. So, please, provide the instructions in comments somewhere within spec-file on how to create this checkout. See this spec, for example: http://cvs.fedoraproject.org/viewvc/rpms/superiotool/devel/superiotool.spec?view=markup +/- The package successfully compiles and builds into binary rpms on at least one primary architecture. Unfortunately, the package failed to build on EL-5 due to missing dependencies on mingw32-{libtiff|proj}. Please, keep this in mind, then you'll requesting cvs branches. + All build dependencies are listed in BuildRequires. 0 No need to handle locales. 0 No shared library files. + The package does NOT bundle copies of system libraries. + The package is not designed to be relocatable. + The package owns all directories that it creates. + The package does not list a file more than once in the spec file's %files listings. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + The package consistently uses macros. + The package contains code, or permissible content. 0 No extremely large documentation files. + Anything, the package includes as %doc, does not affect the runtime of the application. + The package does NOT contain any .la libtool archives. 0 Not a GUI application. - The package must NOT own files or directories already owned by other packages. Unfortunately, it owns %{_mingw32_libdir}/pkgconfig which is already owned by mingw32-filesystem. + At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + All filenames in rpm packages are valid UTF-8 Please, fixissues noted above, and I'll continue.
Ping, Dave.
Sorry, was out of town. Will get to this shortly.
+/- The package is named according to the Package Naming Guidelines, except one small issue in the versioning scheme - since you're packaging pre-release (from VCS), then "Release" field should start from 0. Done +/- The sources used to build the package, must match the upstream source, as provided in the spec URL. However, since this is a pre-release and no tarball was provided by upstream, the only way to check the integrity of the sources, is to make diff against freshly checked out VCS tree. So, please, provide the instructions in comments somewhere within spec-file on how to create this checkout. Done +/- The package successfully compiles and builds into binary rpms on at least one primary architecture. Unfortunately, the package failed to build on EL-5 due to missing dependencies on mingw32-{libtiff|proj}. Please, keep this in mind, then you'll requesting cvs branches. Noted - The package must NOT own files or directories already owned by other packages. Unfortunately, it owns %{_mingw32_libdir}/pkgconfig which is already owned by mingw32-filesystem. Fixed SPEC: http://adsllc.fedorapeople.org/rpmbuild/SPECS/mingw32-libgeotiff.spec Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1813287
Ok, I don't see any other issues, so this package is APPROVED.
Thanks Peter! New Package CVS Request ======================= Package Name: mingw32-libgeotiff Short Description: MinGW port of libgeotiff Georeferenced image library Owners: adsllc Branches: F-11 F-12
CVS done.
Ping, Dave. We are waiting for you :)
Thanks for the ping... I now seem to be stuck. I've gotten to "Tag Or Update Your Branches" on this list: http://fedoraproject.org/wiki/PackageMaintainers/Join But I'm getting this error: [davel@myth F-12]$ make tag rpm: no arguments given for query cvs tag -c mingw32-libgeotiff-- ERROR: Tag mingw32-libgeotiff-- is not in name-version-release format cvs tag: Pre-tag check failed cvs [tag aborted]: correct the above errors first! make: *** [tag] Error 1 I've never seen this with any previous packages - suggestions?
It seems that you forgot to checkout spec-file first. $ ./common/cvs-import.sh -b "branch name" -m "message" ~/path/to/mingw32-libgeotiff.srpm
I had done ./common/cvs-import.sh ~/path/to/mingw32-libgeotiff.srpm. To be safe, I just did ./common/cvs-import.sh -b "F-12" -m "Initial F-12 import" /home/davel/rpmbuild/SRPMS/mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12.src.rpm and ./common/cvs-import.sh -b "F-11" -m "Initial F-11 import" /home/davel/rpmbuild/SRPMS/mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12.src.rpm per your suggestion. I still get the same error.
Perhaps you forgot to type $ cvs up in the mingw32-libgeotiff cvs directory. Since my last comment I see that you're uploaded all necessary files, and it's time to download them locally :)
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/mingw32-libgeotiff-1.3.0-0.1.svn1664.fc11
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12
Thanks Peter. You were spot-on. I swear I tried that before.
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update mingw32-libgeotiff'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-12480
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update mingw32-libgeotiff'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2009-12490
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
mingw32-libgeotiff-1.3.0-0.1.svn1664.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.