Spec URL: http://people.redhat.com/green/FE/FC5/raptor.spec SRPM URL: http://people.redhat.com/green/FE/FC5/raptor-1.4.8-2.src.rpm Description: Raptor is the RDF Parser Toolkit for Redland that provides a set of standalone RDF parsers, generating triples from RDF/XML or N-Triples. Ardour is dependent on this package. The spec file is based on a very old planet ccrma spec file.
Created attachment 127980 [details] some spec fixes Attached patch includes some fixes. | checking GRDDL parser requirements... no - libxml2 and libxslt | are both not available It's missing "BuildRequires: libxslt-devel". * BR curl libxml2 are redundant, because of BR curl-devel libxml2-devel * ASL2? While the old Apache Software License 1.1 maybe was called ASL by some people, better give the new "Apache Licence, Version 2.0" its full name, so "Licence: LGPL or Apache License Version 2.0" ( http://www.opensource.org/licenses/apache2.0.php ) * No reason not to use the %configure macro. * The following two are a major problem IMO: $ raptor-config --libs -L/usr/lib -lraptor -L/usr/kerberos/lib -lcurl -ldl -lgssapi_krb5 -lkrb5 -lk5crypto -lkrb5support -lcom_err -lresolv -lidn -lssl -lcrypto -lz -L/usr/lib -lxml2 -lz -lm -L/usr/lib -lxslt -lxml2 -lz -lm $ pkg-config --libs raptor -L/usr/kerberos/lib -lraptor -lcurl -ldl -lgssapi_krb5 -lkrb5 -lk5crypto -lkrb5support -lcom_err -lresolv -lidn -lssl -lcrypto -lxslt -lxml2 -lz -lm The results of both commands imply that you need much more than just "Requires: libxml2-devel" inside the raptor-devel package. Else linking against libraptor would fail, and developers/packagers would need to add the missing -devel package dependencies manually. Painful.
(In reply to comment #1) > Created an attachment (id=127980) [edit] > some spec fixes Thanks Micheal. I'll try to post new bits tomorrow with your changes and more.
Here are the updated bits, including Michael's spec file patch and additional changes to address his other concerns... Spec URL: http://people.redhat.com/green/FE/FC5/raptor.spec SRPM URL: http://people.redhat.com/green/FE/FC5/raptor-1.4.8-3.src.rpm
Created attachment 128132 [details] patch against raptor-1.4.8 I believe this patch to be the clean fix when linking dynamically. Without it, the pkgconfig file creates RPM dependency hell. With this patch applied, liblrdf really needs only "BuildRequires: pkgconfig raptor-devel ladspa-devel" and not openssl-devel curl2-devel glib2-devel zlib-devel e2fsprogs-devel krb5-devel (and raptor-devel doesn't need them anymore either).
(In reply to comment #4) > Created an attachment (id=128132) [edit] > patch against raptor-1.4.8 > > I believe this patch to be the clean fix when linking dynamically. Thanks. Updated bits here: Spec URL: http://people.redhat.com/green/FE/FC5/raptor.spec SRPM URL: http://people.redhat.com/green/FE/FC5/raptor-1.4.8-4.src.rpm I've updated liblrdf based on this as well.
Created attachment 128276 [details] a patch for raptor-config I've started taking a first look at the other pending packages (jack, ardour,...), but here more rpm dependency hell is left: /usr/bin/raptor-config also includes lots of linker commands in its --libs option, which create unnecessary dependencies on many -devel packages. It would benefit from a patch just as done with the pkg-config file (see attachment). Then, package raptor-devel does not depend on all these -devel packages anymore when linking dynamically, and you can do the following to get rid of all the packages you don't depend on directly: --- raptor.spec.orig 2006-04-24 13:12:03.000000000 +0200 +++ raptor.spec 2006-04-27 00:31:30.000000000 +0200 @@ -19,8 +19,6 @@ Summary: Libraries, includes etc to develop with Raptor RDF parser library Group: Development/Libraries Requires: %{name} = %{version}-%{release} -Requires: libxml2-devel, curl-devel, krb5-devel, libidn-devel, openssl-devel -Requires: zlib-devel, libxslt-devel, e2fsprogs-devel %description devel Libraries, includes etc to develop with Raptor RDF parser library.
And after these changes (which you can apply in CVS) I have no further comments, as the package looks clean then and liblrdf built fine with it, too. APPROVED
(In reply to comment #7) > APPROVED Thanks!
I'd be very thankful if you could request and maintain a EPEL-5 branch for this package. In case you don't want or can't do that, let me know and I'll do that. Thanks!
Maintainer is OK with the change as per previous conversation with him. Package Change Request ====================== Package Name: raptor New Branches: EL-5 Owners for new branch: green,lkundrak cvsextras commits for new branch: yes
cvs done.