Bug 226197 - Merge Review: nfs-utils-lib
Merge Review: nfs-utils-lib
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Steve Dickson
Fedora Package Reviews List
: Reopened
Depends On: 483464
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 15:16 EST by Nobody's working on this, feel free to take it
Modified: 2013-06-15 00:41 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2013-06-15 00:41:50 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:16:27 EST
Fedora Merge Review: nfs-utils-lib

http://cvs.fedora.redhat.com/viewcvs/devel/nfs-utils-lib/
Initial Owner: steved@redhat.com
Comment 1 Michael Schwendt 2009-04-01 04:33:46 EDT
* License:

libnfsidmap is "BSD".

librpcseggss is "BSD" and "Sun RPC", which is a "bad license" according to the Fedora Licensing Guidelines:

http://fedoraproject.org/wiki/Licensing#Bad_Licenses
https://fedoraproject.org/wiki/Licensing/SunRPC


* Static libraries must be moved into a -static subpackage
according to the guidelines:
https://fedoraproject.org/wiki/Packaging:Guidelines#Packaging_Static_Libraries


> Requires(postun): /sbin/ldconfig
> Requires(pre): /sbin/ldconfig

These are automatically added by rpmbuild, because -p /sbin/ldconfig is specified for the scriptlets.


> Requires: libgssglue, openldap

These ought to be removed in favour of the automatically added SONAME dependencies.
Comment 2 Tom "spot" Callaway 2009-05-20 13:43:21 EDT
The SunRPC licensing is now replaced entirely by the BSD license, with explicit permission from Sun.
Comment 3 Tom "spot" Callaway 2009-06-10 09:36:17 EDT
Reopening. No one did a review here!
Comment 4 Steve Dickson 2009-06-10 09:46:18 EDT
oops... sorry about that... I though this was about the licensing problems...
Comment 5 Michael Schwendt 2009-06-10 10:03:57 EDT
Well, there are some good review findings in the bottom half of comment 1. To add to that, the "Requires(pre): /sbin/ldconfig" is completely superfluous because the packages doesn't contain a pre scriptlet at all.
Comment 6 Jason Tibbitts 2013-06-15 00:41:50 EDT
This package has been dead for some time now, replaced by libnfsidmap, but it was never blocked or marked as a dead package.  In any case, there's no point in having this ticket open.

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