Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 481774 - sys-fs/cryptsetup-1.6.2 should not depend on dev-libs/libgpg-error[static-libs(+)] sys-libs/e2fsprogs-libs[static-libs(+)]
Summary: sys-fs/cryptsetup-1.6.2 should not depend on dev-libs/libgpg-error[static-lib...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-20 18:12 UTC by Sven E.
Modified: 2014-11-01 01:03 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sven E. 2013-08-20 18:12:04 UTC
cryptsetup pulls in static libs for othr packages, no matter if they are needed.

there is a hard dep on libgpg-error even for -gcrypt, while cryptetup does not use anything fro libgpgerror.

Same holds for e2fsprogslibs, crptsetup dos no use anything from there (might be use flag dependant, but if so, the dep should depend on that particular flag)

Reproducible: Always

Steps to Reproduce:
1. try building static cryptsetup
2.
3.
Actual Results:  
unused deps are enforced

Expected Results:  
build with what's really needed
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2013-08-21 13:42:18 UTC
It's unclear what changes you want to see. Maybe you could attach a patch with your proposed changes.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2013-08-21 14:25:46 UTC
I browsed quickly through sourcetree of cryptsetup-1.6.2, and when static cryptsetup is built, the configure.ac adds -lgpg-error to LIBS
However when dynamic build is enabled, -lgpg-error isn't added to LIBS
So looks like upstream is including -lgpg-error in LIBS just because some other library uses it to help the static build

And couldn't find any reference to libss, libcom_error, lib... from e2fsprogs-libs, not even for static build

So seems like dropping whole sys-libs/e2fsprogs-libs[static-libs(+)] and whole dev-libs/libgpg-error[static-libs(+)] from the ebuild seems like the correct approach, at most, leave
static? ( dev-libs/libgpg-error[static-libs(+)] )
to the ebuild, and no e2fsprogs at all
Comment 3 Sven E. 2013-08-21 22:45:46 UTC
I can't seem to find any usage of libgpgerror in the source (my best guess is it might be used for gcrypt backend)

Is there any straighforward way to check on this? I.E.: modify source and try a build to check for missing references during linkage?

Another thing, shouldn't ldd for a dynamic executeable list these dependencies aswell?

(In reply to Samuli Suominen from comment #2)
> I browsed quickly through sourcetree of cryptsetup-1.6.2, and when static
> cryptsetup is built, the configure.ac adds -lgpg-error to LIBS
> However when dynamic build is enabled, -lgpg-error isn't added to LIBS
> So looks like upstream is including -lgpg-error in LIBS just because some
> other library uses it to help the static build
> 
> And couldn't find any reference to libss, libcom_error, lib... from
> e2fsprogs-libs, not even for static build
> 
> So seems like dropping whole sys-libs/e2fsprogs-libs[static-libs(+)] and
> whole dev-libs/libgpg-error[static-libs(+)] from the ebuild seems like the
> correct approach, at most, leave
> static? ( dev-libs/libgpg-error[static-libs(+)] )
> to the ebuild, and no e2fsprogs at all
Comment 4 SpanKY gentoo-dev 2013-11-30 18:27:31 UTC
(In reply to Samuli Suominen from comment #2)

i feel like i did a similar analysis at some point and came to same conclusion (re-overlinking)

for e2fsprogs, that might be due to uuid usage ?  it's been in there for quite a long time too, so it might just be outdated now.