[Bug 378841] Review Request: python-rdflib - Python library for working with RDF

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Ruben Kerkhof <ruben at rubenkerkhof.com>  2008-08-24 07:12:38 EDT ---
Hi Dave,

A few comments:
- You could probably remove the conditional %if 0%{?fedora} >= 8 since older
versions are not supported anymore
- Permissions on SPARQLParserc.so are 0775 but should be 0755.
- Rpm adds SPARQLParserc.so to its provides list, but that library is not in a
standard library location, and never meant to be used by other software. It
probably can't do much harm, but it might be nice to override rpm's provide
script for that, to avoid issues when another package decides to name a library
SPARQLParserc.so

- For the other rpmlint warnings, you could either make those files executable
or strip out the shebang line during install.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.