|Summary:||Merge Review: nfs-utils-lib|
|Product:||[Fedora] Fedora||Reporter:||Nobody's working on this, feel free to take it <nobody>|
|Component:||Package Review||Assignee:||Steve Dickson <steved>|
|Status:||CLOSED NOTABUG||QA Contact:||Fedora Package Reviews List <fedora-package-review>|
|Version:||rawhide||CC:||alexl, bugs.michael, steved, tcallawa|
|Fixed In Version:||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2013-06-15 04:41:50 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Bug Depends On:||483464|
Description Nobody's working on this, feel free to take it 2007-01-31 20:16:27 UTC
Fedora Merge Review: nfs-utils-lib http://cvs.fedora.redhat.com/viewcvs/devel/nfs-utils-lib/ Initial Owner: firstname.lastname@example.org
Comment 1 Michael Schwendt 2009-04-01 08:33:46 UTC
* 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 17:43:21 UTC
The SunRPC licensing is now replaced entirely by the BSD license, with explicit permission from Sun.
Comment 3 Tom "spot" Callaway 2009-06-10 13:36:17 UTC
Reopening. No one did a review here!
Comment 4 Steve Dickson 2009-06-10 13:46:18 UTC
oops... sorry about that... I though this was about the licensing problems...
Comment 5 Michael Schwendt 2009-06-10 14:03:57 UTC
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 04:41:50 UTC
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.