[Bug 241403] Review Request: qgis - A user friendly Open Source Geographic Information System

bugzilla at redhat.com bugzilla at redhat.com
Fri Jun 22 17:52:37 UTC 2007


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: qgis - A user friendly Open Source Geographic Information System


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





------- Additional Comments From silfreed at silfreed.net  2007-06-22 13:52 EST -------
(In reply to comment #20)
> * rpm call
>   - Interal rpm call in rpmbuild is forbidden because it is
>     considered as dangerous due to some reason.
>     Please change the following to the proper way.
> -----------------------------------------------------------
> %define grass_ver %(rpm -q --queryformat %{VERSION} grass)
> -----------------------------------------------------------

Sorry; what would the "proper way" be?  The cmake script tries to lookup the 
grass library in the wrong locations and I need to tell it that the library 
lives under a directory like /usr/lib/grass-6.2.1.

> * non-sover libraries with providing -devel subpackage.

Upstream does not provide a sover with their library but their program links 
against it:
$ ldd /usr/bin/qgis | grep libqgis_core.so
        libqgis_core.so => /usr/lib/libqgis_core.so (0x03099000)
$ ldd /usr/lib/qgis/libgrass*.so | grep libqgisgrass.so
        libqgisgrass.so => /usr/lib/libqgisgrass.so (0x00673000)
        libqgisgrass.so => /usr/lib/libqgisgrass.so (0x002fc000)

Should I still move these files into the -devel package and then have the main 
packages end up with a Requires on the -devel package, or do I need to work 
with upstream to add a sover to these libraries?

-- 
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