Bug 189309

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.