Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 995688 - Remove bundled libraries from davix
Summary: Remove bundled libraries from davix
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: davix
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Adrien Devresse
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: DuplicSysLibsTracker
TreeView+ depends on / blocked
 
Reported: 2013-08-10 05:59 UTC by Mattias Ellert
Modified: 2013-11-06 13:46 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-11-06 13:46:51 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Mattias Ellert 2013-08-10 05:59:36 UTC
Description of problem:

The davix source tree contains many libraries that should be unbundled according to the guidelines.

https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries

deps/libneon/

 → Use BuildRequires: neon-devel instead

deps/strlcpy/

 → Use BuildRequires: libbsd-devel instead
   #include <bsd/string.h>
   -lbsd

test/gtest-1.6.0/

 → Use BuildRequires: gtest-devel instead

test/pywebdav/

 → Use BuildRequires; pywebdav instead

Other code in the deps/* directories should be investigated to assess whether they are bundled or not.

Unused bundled code should be deleted in %prep so that it is no accidentally used during the build.

Version-Release number of selected component (if applicable):

davix-0.2.2-2


Additional info:

There are some other minor packaging issues in the spec as well:

The main davix package depends on the davix-libs package and these are build from the same srpm → the davix package should have a fully versioned dependency on davix-libs:

Requires: %{name}-libs%{?_isa} = %{version}-%{release}

The doc subpackage should be noarch where supported, i.e. except on EPEL 5:

%if %{?fedora}%{!?fedora:0} >= 10 || %{?rhel}%{!?rhel:0} >= 6
BuildArch:	noarch
%endif

The doc package depends on the main package, but contains documentation about the api of the library, so this dependency does not make sense. It could depend on the library package instead, or have no dependency at all. (If the dependency is removed the LICENSE file should be added to the doc package.)

Comment 1 Adrien Devresse 2013-08-12 14:30:53 UTC
Hi Matthias,

Thank you for your bug report and your remarks.


But please keep in mind that davix has been packaged in a preview and beta stage for user feedback purpose ( version < 1 for now ) and most of the problems you are refering too are under corrections or already corrected upstream.

However concerning libneon, even if the name leads to confusion, this is not a bundle library : davix can't compile with a standard libneon dependency. 

The src code inside deps/libneon has been heavily modified for VOMS, grid specific authentication support ( PEM, chains, etc.. ), grid extensions supports, S3 etc ... and can't be considered as "neon" anymore and will even more derivate from neon in future.

These modifications can not be integrated upstreams too, they derivate completely from the original libneon aim and are grid specific.

However and if needed because of any name conflict, we can proceed to the renaming of the source code of this component.


Regards,
Adrien

Comment 2 Adrien Devresse 2013-09-17 08:14:27 UTC
The changes for this problems have been applied and are now deployed under EPEL testing with the 0.2.4 version.


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