Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package nfs-utils for openSUSE:Factory 
checked in at 2021-10-29 22:33:36
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/nfs-utils (Old)
 and      /work/SRC/openSUSE:Factory/.nfs-utils.new.1890 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "nfs-utils"

Fri Oct 29 22:33:36 2021 rev:166 rq:927402 version:unknown

Changes:
--------
--- /work/SRC/openSUSE:Factory/nfs-utils/nfs-utils.changes      2021-09-13 
16:24:55.978772583 +0200
+++ /work/SRC/openSUSE:Factory/.nfs-utils.new.1890/nfs-utils.changes    
2021-10-29 22:34:26.279686760 +0200
@@ -1,0 +2,9 @@
+Mon Oct 25 23:53:37 UTC 2021 - Neil Brown <nfbr...@suse.com>
+
+- add Add-disable-sbin-override-for-when-sbin-is-a-symlink.patch
+  and use it to move last few binaries from /sbin to /usr/sbin
+  (bsc#1191082)
+- Move nfsdclddb and nfsdclnts from nfs-client to
+  nfs-kernel-server and fix their permissions.
+
+-------------------------------------------------------------------

New:
----
  Add-disable-sbin-override-for-when-sbin-is-a-symlink.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ nfs-utils.spec ++++++
--- /var/tmp/diff_new_pack.9BxIQI/_old  2021-10-29 22:34:26.887686990 +0200
+++ /var/tmp/diff_new_pack.9BxIQI/_new  2021-10-29 22:34:26.887686990 +0200
@@ -43,6 +43,7 @@
 Source27:       nfs-kernel-server.tmpfiles.conf
 Patch0:         nfs-utils-1.0.7-bind-syntax.patch
 Patch1:         0001-gssd-fix-crash-in-debug-message.patch
+Patch2:         Add-disable-sbin-override-for-when-sbin-is-a-symlink.patch
 
 BuildRequires:  e2fsprogs-devel
 BuildRequires:  gcc-c++
@@ -141,6 +142,7 @@
        --enable-mount \
        --enable-libmount-mount \
        --disable-static \
+       --disable-sbin-override \
        --with-pluginpath=%{_libdir}/libnfsidmap-1.0.0 \
        --enable-mountconfig
 make %{?_smp_mflags}
@@ -206,7 +208,7 @@
 ###
 %{fillup_only -n nfs nfs}
 #
-%set_permissions /sbin/mount.nfs
+%set_permissions %{_sbindir}/mount.nfs
 /sbin/ldconfig
 %service_add_post auth-rpcgss-module.service nfs-idmapd.service 
nfs-blkmap.service rpc-statd-notify.service rpc-gssd.service rpc-statd.service 
rpc-svcgssd.service
 
@@ -218,7 +220,7 @@
 %service_del_postun auth-rpcgss-module.service nfs-idmapd.service 
nfs-blkmap.service rpc-statd-notify.service rpc-gssd.service rpc-statd.service 
rpc-svcgssd.service
 
 %verifyscript -n nfs-client
-%verify_permissions -e /sbin/mount.nfs
+%verify_permissions -e %{_sbindir}/mount.nfs
 
 %pre -n nfs-kernel-server
 %service_add_pre nfs-svcgssd.service nfs-mountd.service nfs-server.service
@@ -254,14 +256,12 @@
 %config %{_sysconfdir}/idmapd.conf
 %config %{_sysconfdir}/nfsmount.conf
 %config %{_sysconfdir}/nfs.conf
-%verify(not mode) %attr(0755,root,root) /sbin/mount.nfs
-/sbin/mount.nfs4
-/sbin/umount.nfs
-/sbin/umount.nfs4
+%verify(not mode) %attr(0755,root,root) %{_sbindir}/mount.nfs
+%{_sbindir}/mount.nfs4
+%{_sbindir}/umount.nfs
+%{_sbindir}/umount.nfs4
 %attr(0755,root,root) %{_sbindir}/mountstats
 %attr(0755,root,root) %{_sbindir}/nfsiostat
-%{_sbindir}/nfsdclddb
-%{_sbindir}/nfsdclnts
 %{_sbindir}/nfsdcld
 %{_sbindir}/nfsidmap
 %{_sbindir}/nfsstat
@@ -343,7 +343,9 @@
 %{_sbindir}/rcnfs-server
 %{_sbindir}/rpc.mountd
 %{_sbindir}/rpc.nfsd
-/sbin/nfsdcltrack
+%{_sbindir}/nfsdcltrack
+%attr(0755,root,root) %{_sbindir}/nfsdclddb
+%attr(0755,root,root) %{_sbindir}/nfsdclnts
 %{_mandir}/man5/exports.5%{ext_man}
 %{_mandir}/man7/nfsd.7%{ext_man}
 %{_mandir}/man8/exportfs.8%{ext_man}

++++++ Add-disable-sbin-override-for-when-sbin-is-a-symlink.patch ++++++
>From 49822c2130a2a6e1d5a9e414f5a0726a461d9b00 Mon Sep 17 00:00:00 2001
From: NeilBrown <ne...@suse.de>
Date: Mon, 25 Oct 2021 12:08:46 -0400
Subject: [PATCH] Add --disable-sbin-override for when /sbin is a symlink

mount.nfs* umount.nfs* and nfsdcltrack are currently always installed in
/sbin.

Many distros are moving to a "merged /usr" where /sbin and others are
symlinks into /usr/sbin or similar.  In these cases it is inelegant to
install in /sbin (i.e. install through a symlink).

So we add "--disable-sbin-override" as a configure option.  This causes
the same sbindir to be used for *mount.nfs* and nfsdcltrack as for other
system binaries.

Note that autotools notices if we simply define "sbindir=/sbin"
inside an "if CONFIG_foo" clause, gives a warning, and defeats our
intent.

So instead, we use the @CONFIG_SBIN_OVERRIDE_TRUE@ prefix to find
the new declaration when we don't want it.

Signed-off-by: NeilBrown <ne...@suse.de>
Signed-off-by: Steve Dickson <ste...@redhat.com>
---
 configure.ac                  | 6 ++++++
 utils/mount/Makefile.am       | 8 +++++---
 utils/nfsdcltrack/Makefile.am | 9 ++++++---
 3 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/configure.ac b/configure.ac
index bc2d0f02979c..93626d62be40 100644
--- a/configure.ac
+++ b/configure.ac
@@ -187,6 +187,12 @@ else
        enable_libmount=no
 fi
 
+AC_ARG_ENABLE(sbin-override,
+       [AC_HELP_STRING([--disable-sbin-override],
+               [Don't force nfsdcltrack and mount helpers into /sbin: always 
honour --sbindir])],
+       enable_sbin_override=$enableval,
+       enable_sbin_override=yes)
+       AM_CONDITIONAL(CONFIG_SBIN_OVERRIDE, [test "$enable_sbin_override" = 
"yes"])
 AC_ARG_ENABLE(junction,
        [AC_HELP_STRING([--enable-junction],
                        [enable support for NFS junctions @<:@default=no@:>@])],
diff --git a/utils/mount/Makefile.am b/utils/mount/Makefile.am
index ad0be93b1def..3101f7abd7f4 100644
--- a/utils/mount/Makefile.am
+++ b/utils/mount/Makefile.am
@@ -1,8 +1,10 @@
 ## Process this file with automake to produce Makefile.in
 
-# These binaries go in /sbin (not /usr/sbin), and that cannot be
-# overridden at config time.
-sbindir = /sbin
+# These binaries go in /sbin (not /usr/sbin), unless CONFIG_SBIN_OVERRIDE
+# is disabled as may be appropriate when /sbin is a symlink.
+# Note that we don't use "if CONFIG_SBIN_OVERRIDE" as that
+# causes autotools to notice the override and disable it.
+@CONFIG_SBIN_OVERRIDE_TRUE@sbindir = /sbin
 
 man8_MANS      = mount.nfs.man umount.nfs.man
 man5_MANS      = nfs.man
diff --git a/utils/nfsdcltrack/Makefile.am b/utils/nfsdcltrack/Makefile.am
index 2f7fe3de6922..769e4a455fcf 100644
--- a/utils/nfsdcltrack/Makefile.am
+++ b/utils/nfsdcltrack/Makefile.am
@@ -1,8 +1,11 @@
 ## Process this file with automake to produce Makefile.in
 
-# These binaries go in /sbin (not /usr/sbin), and that cannot be
-# overridden at config time. The kernel "knows" the /sbin name.
-sbindir = /sbin
+# These binaries go in /sbin (not /usr/sbin) as the kernel "knows" the
+# /sbin name.  If /sbin is a symlink, CONFIG_SBIN_OVERRIDE can be
+# disabled to install in /usr/sbin anyway.
+# Note that we don't use "if CONFIG_SBIN_OVERRIDE" as that
+# causes autotools to notice the override and disable it.
+@CONFIG_SBIN_OVERRIDE_TRUE@sbindir = /sbin
 
 man8_MANS      = nfsdcltrack.man
 EXTRA_DIST     = $(man8_MANS)
-- 
2.33.1

Reply via email to