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

bugzilla at redhat.com bugzilla at redhat.com
Tue May 29 18:09:56 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-05-29 14:09 EST -------
Spec URL: http://www.silfreed.net/download/repo/packages/qgis/qgis.spec
SRPM URL: 
http://www.silfreed.net/download/repo/packages/qgis/qgis-0.8.0-5.src.rpm

Changelog:
* Tue May 29 2007 Douglas E. Warner <silfreed at silfreed.net> 0.8.0-5
- fixing more directory owernship (themes, themes-default)
- fixing qt4 hardcoded lib path
- removing Requires ldconfig
- adding BuildRequires sqlite-devel
- adding patch for supporting python 2.5 in configure script


(In reply to comment #4)
> * Mockbuild failed as attached. First check BuildRequires.
> * Requires: /sbin/ldconfig is not needed.
> * Some directories are still not owned
>   (e.g. %{_datadir}/%{name}/themes/)

I removed the Requires for ldconfig and fixed directory ownership 
for %{_datadir}/%{name}/themes/ and %{_datadir}/%{name}/themes/default/.

(In reply to comment #5)
> Use %{_libdir} instead of /usr/lib. on x86_64/ppc64 arch,
> qt4 moc (and others) are under /usr/lib64/qt4/bin/ 

Fixed.  This seems to fix the rpmlint error as well.


$ rpmlint -v RPMS/qgis-*0.8.0-5*.rpm SRPMS/qgis-0.8.0-5.src.rpm
I: qgis checking
I: qgis-debuginfo checking
I: qgis-devel checking
W: qgis-devel no-documentation
I: qgis-grass checking
W: qgis-grass no-documentation
I: qgis-theme-nkids checking
W: qgis-theme-nkids no-documentation
I: qgis checking


-- 
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 package-review mailing list