Bug 189309
Summary: | Review Request: raptor | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Anthony Green <green> | ||||||||
Component: | Package Review | Assignee: | Michael Schwendt <bugs.michael> | ||||||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||||||
Severity: | medium | Docs Contact: | |||||||||
Priority: | medium | ||||||||||
Version: | rawhide | CC: | lkundrak | ||||||||
Target Milestone: | --- | Flags: | kevin:
fedora-cvs+
|
||||||||
Target Release: | --- | ||||||||||
Hardware: | All | ||||||||||
OS: | Linux | ||||||||||
Whiteboard: | |||||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||||
Doc Text: | Story Points: | --- | |||||||||
Clone Of: | Environment: | ||||||||||
Last Closed: | 2006-04-27 14:44:37 UTC | Type: | --- | ||||||||
Regression: | --- | Mount Type: | --- | ||||||||
Documentation: | --- | CRM: | |||||||||
Verified Versions: | Category: | --- | |||||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||||
Embargoed: | |||||||||||
Bug Depends On: | |||||||||||
Bug Blocks: | 163779, 189313, 189315 | ||||||||||
Attachments: |
|
Description
Anthony Green
2006-04-19 02:09:19 UTC
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. |