Bug 1082763 - include userspace tools/lib/lockdep in kernel-tools*
Summary: include userspace tools/lib/lockdep in kernel-tools*
Keywords:
Status: ASSIGNED
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kernel Maintainer List
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 1082816 1084680 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-03-31 18:56 UTC by Frank Ch. Eigler
Modified: 2018-10-24 21:29 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)
lockdep build/package for rawhide kernel.spec (1.37 KB, patch)
2014-03-31 18:56 UTC, Frank Ch. Eigler
no flags Details | Diff

Description Frank Ch. Eigler 2014-03-31 18:56:56 UTC
Created attachment 881041 [details]
lockdep build/package for rawhide kernel.spec

The userspace lockdep library, though early, would still be nice to package.

Comment 1 Josh Boyer 2014-03-31 20:13:12 UTC
A few comments/questions.

I really dislike shipping unversioned .so libraries as the main deliverable.  That provides no mechanism to avoid ABI/API conflicts in applications using the library.  As you said, it's pretty early...

Why would we ship liblockdep.a?  That's going to require a static lib allowance from FESCo and seems unnecessary.

Why wouldn't we ship the lockdep wrapper?

We had a similar request (though I can't find a link to it at the moment) for libtraceevent.so and we said we were going to wait for a versioned shared library first.  I would guess this falls into the same category.

Comment 2 Frank Ch. Eigler 2014-03-31 20:20:11 UTC
> I really dislike shipping unversioned .so libraries as the main deliverable.

Understood; shall we improvise a git-describe type version?

> Why would we ship liblockdep.a?  That's going to require a static lib
> allowance from FESCo and seems unnecessary.

Good point.

> Why wouldn't we ship the lockdep wrapper?

It is in %{_bindir} .. though it probably doesn't work as is
due to LD_PRELOAD="./liblockdep.so" hardcoding therein.  Maybe
create a new one during %build/%install, based on the improvised
versioned soname?

Comment 3 Josh Boyer 2014-03-31 21:24:53 UTC
I'd rather upstream actually create a proper shared library and use -soname with a proper version string.  Then liblockdep.so becomes a symlink in the -devel package, like every other shared library.  I'll bug them about it.

Comment 4 Josh Boyer 2014-03-31 21:25:53 UTC
*** Bug 1082816 has been marked as a duplicate of this bug. ***

Comment 6 Robin Hack 2014-04-09 07:34:47 UTC
Hi guys. I tried create whole separate package of liblockdep in bug bz#1084680. Should I close that bug as a duplicity?

Comment 7 Josh Boyer 2014-04-09 13:54:13 UTC
(In reply to Robin Hack from comment #6)
> Hi guys. I tried create whole separate package of liblockdep in bug
> bz#1084680. Should I close that bug as a duplicity?

Yeah, that's fine.

Comment 8 Robin Hack 2014-04-10 05:24:45 UTC
*** Bug 1084680 has been marked as a duplicate of this bug. ***


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