Bug 189309 - Review Request: raptor
Review Request: raptor
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Michael Schwendt
Fedora Package Reviews List
Depends On:
Blocks: FE-ACCEPT 189313 189315
  Show dependency treegraph
Reported: 2006-04-18 22:09 EDT by Anthony Green
Modified: 2008-04-14 15:34 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2006-04-27 10:44:37 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
kevin: fedora‑cvs+

Attachments (Terms of Use)
some spec fixes (1.69 KB, patch)
2006-04-19 07:36 EDT, Michael Schwendt
no flags Details | Diff
patch against raptor-1.4.8 (395 bytes, patch)
2006-04-23 16:50 EDT, Michael Schwendt
no flags Details | Diff
a patch for raptor-config (464 bytes, patch)
2006-04-26 18:26 EDT, Michael Schwendt
no flags Details | Diff

  None (edit)
Description Anthony Green 2006-04-18 22:09:19 EDT
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
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.
Comment 1 Michael Schwendt 2006-04-19 07:36:43 EDT
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.
Comment 2 Anthony Green 2006-04-19 13:37:03 EDT
(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.
Comment 3 Anthony Green 2006-04-23 07:34:17 EDT
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
Comment 4 Michael Schwendt 2006-04-23 16:50:23 EDT
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).
Comment 5 Anthony Green 2006-04-24 07:08:33 EDT
(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.
Comment 6 Michael Schwendt 2006-04-26 18:26:47 EDT
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

--- 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.
Comment 7 Michael Schwendt 2006-04-26 18:29:33 EDT
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.

Comment 8 Anthony Green 2006-04-27 10:44:37 EDT
(In reply to comment #7)


Comment 9 Lubomir Kundrak 2008-04-11 17:14:52 EDT
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.

Comment 10 Lubomir Kundrak 2008-04-14 13:10:51 EDT
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
Comment 11 Kevin Fenzi 2008-04-14 15:34:59 EDT
cvs done.

Note You need to log in before you can comment on or make changes to this bug.