Bug 1718738
| Summary: | [RHEL8] update the spec file not adding libnfsidmap redundant manpage files under mandir | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 8 | Reporter: | Yongcheng Yang <yoyang> | ||||
| Component: | nfs-utils | Assignee: | Steve Dickson <steved> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Yongcheng Yang <yoyang> | ||||
| Severity: | unspecified | Docs Contact: | |||||
| Priority: | low | ||||||
| Version: | 8.0 | CC: | steved, swhiteho, tdawson, xzhou | ||||
| Target Milestone: | rc | Keywords: | Reproducer | ||||
| Target Release: | 8.1 | Flags: | pm-rhel:
mirror+
|
||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | nfs-utils-2.3.3-24.el8 | Doc Type: | If docs needed, set a value | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2019-11-05 22:18:52 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Yongcheng Yang
2019-06-10 06:34:19 UTC
libnfsidmap is a package supplied by nfs-utils. It originally was it's own seperate package, but was pulled into nfs-utils with this bugzilla. https://bugzilla.redhat.com/show_bug.cgi?id=1611343 As far as both this package, and nfs-utils having the same man page, this is ok technically. Since they are both coming from the same build, they are always going to be the same file. So that is safe. Is it confusing for users? Yes, it can be. The nfs-utils spec file needs to be updated so that nfs-utils has only the man pages relating to it's package, and libnfsidmap has only the man pages for packages that is uses. Thank Troy for the explanation. (In reply to Troy Dawson from comment #1) ... > > The nfs-utils spec file needs to be updated so that nfs-utils has only the > man pages relating to it's package, and libnfsidmap has only the man pages > for packages that is uses. Maybe something like this to let only nfs-utils own the 2 man files. -------------------------------------------- diff --git a/nfs-utils.spec b/nfs-utils.spec index 331d524..6b5d49e 100644 --- a/nfs-utils.spec +++ b/nfs-utils.spec @@ -312,8 +312,6 @@ fi %config(noreplace) %{_sysconfdir}/idmapd.conf %{_libdir}/libnfsidmap.so.* %{_libdir}/libnfsidmap/*.so -%{_mandir}/man3/nfs4_uid_to_name.* -%{_mandir}/man5/idmapd.conf.* %files -n libnfsidmap-devel %{_libdir}/pkgconfig/libnfsidmap.pc -------------------------------------------- Another annoying issue is the mess files under /user/lib/.build-id/ as following. I don't know if they are necessary. If not, better to get rid of them IMHO. (In reply to Yongcheng Yang from comment #0) ... > [root@hp-dl380pg8-09 ~]# rpm -ql libnfsidmap > /etc/idmapd.conf > /usr/lib/.build-id > /usr/lib/.build-id/4a > /usr/lib/.build-id/4a/fc423c29d49d39ac0489a7a38b5a8d544056b3 <<< > /usr/lib/.build-id/62 > /usr/lib/.build-id/62/eb6ef98c1f684ffd18b7d5736544b10964be19 <<< > /usr/lib/.build-id/67 > /usr/lib/.build-id/67/43f63eabd9bd59819ba2b9a50e5a949e9ba27b <<< > /usr/lib/.build-id/e5 > /usr/lib/.build-id/e5/1042f8c512be067e0167ebe7c42e0381289e63 <<< > /usr/lib64/libnfsidmap.so.1 > /usr/lib64/libnfsidmap.so.1.0.0 > /usr/lib64/libnfsidmap/nsswitch.so > /usr/lib64/libnfsidmap/static.so > /usr/lib64/libnfsidmap/umich_ldap.so > /usr/share/doc/libnfsidmap > /usr/share/doc/libnfsidmap/AUTHORS > /usr/share/doc/libnfsidmap/COPYING > /usr/share/doc/libnfsidmap/README > /usr/share/man/man3/nfs4_uid_to_name.3.gz > /usr/share/man/man5/idmapd.conf.5.gz > [root@hp-dl380pg8-09 ~]# Please feel free to modify the title when we're going to fix any of them. (In reply to Yongcheng Yang from comment #2) > Thank Troy for the explanation. > > (In reply to Troy Dawson from comment #1) > ... > > > > The nfs-utils spec file needs to be updated so that nfs-utils has only the > > man pages relating to it's package, and libnfsidmap has only the man pages > > for packages that is uses. > > Maybe something like this to let only nfs-utils own the 2 man files. > -------------------------------------------- > diff --git a/nfs-utils.spec b/nfs-utils.spec > index 331d524..6b5d49e 100644 > --- a/nfs-utils.spec > +++ b/nfs-utils.spec > @@ -312,8 +312,6 @@ fi > %config(noreplace) %{_sysconfdir}/idmapd.conf > %{_libdir}/libnfsidmap.so.* > %{_libdir}/libnfsidmap/*.so > -%{_mandir}/man3/nfs4_uid_to_name.* > -%{_mandir}/man5/idmapd.conf.* > > %files -n libnfsidmap-devel > %{_libdir}/pkgconfig/libnfsidmap.pc > -------------------------------------------- > Whichever package you want to have them is great. The above looks fine. > Another annoying issue is the mess files under /user/lib/.build-id/ as > following. > I don't know if they are necessary. If not, better to get rid of them IMHO. > > (In reply to Yongcheng Yang from comment #0) > ... > > [root@hp-dl380pg8-09 ~]# rpm -ql libnfsidmap > > /etc/idmapd.conf > > /usr/lib/.build-id > > /usr/lib/.build-id/4a > > /usr/lib/.build-id/4a/fc423c29d49d39ac0489a7a38b5a8d544056b3 <<< > > /usr/lib/.build-id/62 > > /usr/lib/.build-id/62/eb6ef98c1f684ffd18b7d5736544b10964be19 <<< > > /usr/lib/.build-id/67 > > /usr/lib/.build-id/67/43f63eabd9bd59819ba2b9a50e5a949e9ba27b <<< > > /usr/lib/.build-id/e5 > > /usr/lib/.build-id/e5/1042f8c512be067e0167ebe7c42e0381289e63 <<< > > /usr/lib64/libnfsidmap.so.1 > > /usr/lib64/libnfsidmap.so.1.0.0 > > /usr/lib64/libnfsidmap/nsswitch.so > > /usr/lib64/libnfsidmap/static.so > > /usr/lib64/libnfsidmap/umich_ldap.so > > /usr/share/doc/libnfsidmap > > /usr/share/doc/libnfsidmap/AUTHORS > > /usr/share/doc/libnfsidmap/COPYING > > /usr/share/doc/libnfsidmap/README > > /usr/share/man/man3/nfs4_uid_to_name.3.gz > > /usr/share/man/man5/idmapd.conf.5.gz > > [root@hp-dl380pg8-09 ~]# > > Please feel free to modify the title when we're going to fix any of them. Unfortunately the .build stuff has to stay. I'm not a big fan of it, and I'm having a hard time finding why we have to keep it, but we do. Created attachment 1581308 [details]
Proposed patch to fix the issue
Thanks for review! I'm updating bug summary and attaching the proposed patch for now. Verified in nfs-utils-2.3.3-24.el8 now. (In reply to Yongcheng Yang from comment #0) > [root@hp-dl380pg8-09 ~]# rpm -ql libnfsidmap nfs-utils | sort | uniq --repeated > /usr/lib/.build-id > /usr/share/man/man3/nfs4_uid_to_name.3.gz > /usr/share/man/man5/idmapd.conf.5.gz > [root@hp-dl380pg8-09 ~]# > [root@hp-dl380pg8-09 ~]# rpm -qf /usr/share/man/man3/nfs4_uid_to_name.3.gz > libnfsidmap-2.3.3-14.el8_0.x86_64 <<<<<<<<<< > nfs-utils-2.3.3-14.el8_0.x86_64 <<<<<<<<<< > [root@hp-dl380pg8-09 ~]# rpm -qf /usr/share/man/man5/idmapd.conf.5.gz > libnfsidmap-2.3.3-14.el8_0.x86_64 <<<<<<<<<< > nfs-utils-2.3.3-14.el8_0.x86_64 <<<<<<<<<< > [root@hp-dl380pg8-09 ~]# # Currently ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [root.0 ~]# rpm -q nfs-utils nfs-utils-2.3.3-24.el8.x86_64 [root.0 ~]# rpm -qf /usr/share/man/man3/nfs4_uid_to_name.3.gz nfs-utils-2.3.3-24.el8.x86_64 <<<<<<<<<<< [root.0 ~]# rpm -qf /usr/share/man/man5/idmapd.conf.5.gz nfs-utils-2.3.3-24.el8.x86_64 <<<<<<<<<<< [root.0 ~]# rpm -ql libnfsidmap /etc/idmapd.conf /usr/lib/.build-id /usr/lib/.build-id/05 /usr/lib/.build-id/05/e2c492ad0a92a484bd9d21b57dd2f76f085dd6 /usr/lib/.build-id/4f /usr/lib/.build-id/4f/19e110058b6985cb2917e9d34c11ceea87b6e8 /usr/lib/.build-id/96 /usr/lib/.build-id/96/a7b44243945f28404c6278acce14a8b37ac132 /usr/lib/.build-id/b2 /usr/lib/.build-id/b2/f435180cc874481b8539b964d6fa34fecafd69 /usr/lib64/libnfsidmap.so.1 /usr/lib64/libnfsidmap.so.1.0.0 /usr/lib64/libnfsidmap/nsswitch.so /usr/lib64/libnfsidmap/static.so /usr/lib64/libnfsidmap/umich_ldap.so /usr/share/doc/libnfsidmap /usr/share/doc/libnfsidmap/AUTHORS /usr/share/doc/libnfsidmap/COPYING usr/share/doc/libnfsidmap/README /[root.0 ~]# rpm -ql libnfsidmap nfs-utils | sort | uniq --repeated /usr/lib/.build-id [root.0 ~]# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:3579 |