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@xxxxxxxxxxxxxxxx> 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review