Hello community,

here is the log from the commit of package sanlock for openSUSE:Factory checked 
in at 2017-04-17 10:22:23
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/sanlock (Old)
 and      /work/SRC/openSUSE:Factory/.sanlock.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "sanlock"

Mon Apr 17 10:22:23 2017 rev:14 rq:487143 version:3.4.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/sanlock/sanlock.changes  2017-03-29 
13:21:49.547750655 +0200
+++ /work/SRC/openSUSE:Factory/.sanlock.new/sanlock.changes     2017-04-17 
10:22:24.399392465 +0200
@@ -1,0 +2,28 @@
+Fri Apr  7 21:53:33 UTC 2017 - [email protected]
+
+- Remove redundant %if guards around %package and scriptlets.
+  They are only really needed for %files.
+- Replace $RPM_ shell vars by modern macros.
+- Trim descriptions a little.
+
+-------------------------------------------------------------------
+Mon Apr  3 15:43:17 UTC 2017 - [email protected]
+
+- spec: fix permissions on config files
+
+-------------------------------------------------------------------
+Mon Apr  3 15:31:43 UTC 2017 - [email protected]
+
+- Use System/Base in fence and reset subpackages
+
+-------------------------------------------------------------------
+Mon Apr  3 07:21:10 UTC 2017 - [email protected]
+
+- Use correct permissions on unit files (bsc#1032046)
+
+-------------------------------------------------------------------
+Wed Mar 29 10:49:35 UTC 2017 - [email protected]
+
+- prerequire groupadd and useradd.
+
+-------------------------------------------------------------------

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

Other differences:
------------------
++++++ sanlock.spec ++++++
--- /var/tmp/diff_new_pack.8o134Z/_old  2017-04-17 10:22:26.343117190 +0200
+++ /var/tmp/diff_new_pack.8o134Z/_new  2017-04-17 10:22:26.347116625 +0200
@@ -53,6 +53,8 @@
 %if 0%{?suse_version} > 1320
 Requires(pre):  group(disk)
 %endif
+Requires(pre):  %{_sbindir}/groupadd
+Requires(pre):  %{_sbindir}/useradd
 Requires:       %{name}-lib = %{version}-%{release}
 Recommends:     logrotate
 Source0:        %{name}-%{version}.tar.xz
@@ -84,8 +86,7 @@
 Group:          Development/Libraries/C and C++
 
 %description    -n libsanlock1
-The %{name}-lib package contains the runtime libraries for sanlock,
-a shared disk lock manager.
+The runtime libraries for sanlock, a shared disk lock manager.
 Hosts connected to a common SAN can use this to synchronize their
 access to the shared disks.
 
@@ -96,9 +97,8 @@
 Requires:       libsanlock1 = %{version}-%{release}
 
 %description    -n python-%name
-The %{name}-python package contains a module that permits applications
-written in the Python programming language to use the interface
-supplied by the sanlock library.
+A module that permits applications written in the Python programming
+language to use the interface supplied by the sanlock library.
 
 %package        devel
 Summary:        Development files for %{name}
@@ -109,30 +109,25 @@
 The %{name}-devel package contains libraries and header files for
 developing applications that use %{name}.
 
-%if %{with_fence_sanlockd}
 %package        -n fence-sanlock
 Summary:        Fence agent using sanlock and wdmd
-Group:          System Environment/Base
+Group:          System/Base
 Requires:       sanlock = %{version}-%{release}
 
 %description    -n fence-sanlock
-The fence-sanlock package contains the fence agent and
-daemon for using sanlock and wdmd as a cluster fence agent.
-%endif
+Fence agent and daemon for using sanlock and wdmd as a cluster fence agent.
 
-%if %{with_sanlk_reset}
 %package -n     sanlk-reset
 Summary:        Host reset daemon and client using sanlock
-Group:          System Environment/Base
+Group:          System/Base
 Requires:       libsanlock1 = %{version}-%{release}
 Requires:       sanlock = %{version}-%{release}
 
 %description -n sanlk-reset
-The sanlk-reset package contains the reset daemon and client.
+The reset daemon and client for sanlock.
 A cooperating host running the daemon can be reset by a host
 running the client, so long as both maintain access to a
 common sanlock lockspace.
-%endif
 
 %prep
 %setup -q
@@ -148,69 +143,69 @@
 %build
 # upstream does not require configure
 # upstream does not support _smp_mflags
-CFLAGS=$RPM_OPT_FLAGS make -C wdmd
-CFLAGS=$RPM_OPT_FLAGS make -C src
-CFLAGS=$RPM_OPT_FLAGS make -C python
+CFLAGS="%{optflags}" make -C wdmd
+CFLAGS="%{optflags}" make -C src
+CFLAGS="%{optflags}" make -C python
 %if %{with_fence_sanlockd}
-CFLAGS=$RPM_OPT_FLAGS make -C fence_sanlock
+CFLAGS="%{optflags}" make -C fence_sanlock
 %endif
 %if %{with_sanlk_reset}
-CFLAGS=$RPM_OPT_FLAGS make -C reset
+CFLAGS="%{optflags}" make -C reset
 %endif
 
 %install
 make -C src \
         install LIBDIR=%{_libdir} \
-        DESTDIR=$RPM_BUILD_ROOT
+        DESTDIR="%{buildroot}"
 make -C wdmd \
         install LIBDIR=%{_libdir} \
-        DESTDIR=$RPM_BUILD_ROOT
+        DESTDIR="%{buildroot}"
 make -C python \
         install LIBDIR=%{_libdir} \
-        DESTDIR=$RPM_BUILD_ROOT PREFIX=%_prefix
+        DESTDIR="%{buildroot}" PREFIX=%_prefix
 %if %{with_fence_sanlockd}
 make -C fence_sanlock \
         install LIBDIR=%{_libdir} \
-        DESTDIR=$RPM_BUILD_ROOT
+        DESTDIR="%{buildroot}"
 %endif
 %if %{with_sanlk_reset}
 make -C reset \
         install LIBDIR=%{_libdir} \
-        DESTDIR=$RPM_BUILD_ROOT
+        DESTDIR="%{buildroot}"
 %endif
 
-install -D -m 755 src/sanlock.conf 
$RPM_BUILD_ROOT/%{_sysconfdir}/sanlock/sanlock.conf
-install -D -m 755 %SOURCE2 
$RPM_BUILD_ROOT/%{_localstatedir}/adm/fillup-templates/sysconfig.sanlock
-install -D -m 755 %SOURCE4 
$RPM_BUILD_ROOT/%{_localstatedir}/adm/fillup-templates/sysconfig.wdmd
+install -D -m 644 src/sanlock.conf 
%{buildroot}/%{_sysconfdir}/sanlock/sanlock.conf
+install -D -m 644 %SOURCE2 
%{buildroot}/%{_localstatedir}/adm/fillup-templates/sysconfig.sanlock
+install -D -m 644 %SOURCE4 
%{buildroot}/%{_localstatedir}/adm/fillup-templates/sysconfig.wdmd
 
 %if %{with_systemd}
-install -D -m 755 init.d/sanlock.service 
$RPM_BUILD_ROOT/%{_unitdir}/sanlock.service
+install -D -m 644 init.d/sanlock.service 
%{buildroot}/%{_unitdir}/sanlock.service
 ln -s /usr/sbin/service %{buildroot}%{_sbindir}/rcsanlock
-install -D -m 755 init.d/wdmd.service $RPM_BUILD_ROOT/%{_unitdir}/wdmd.service
+install -D -m 644 init.d/wdmd.service %{buildroot}/%{_unitdir}/wdmd.service
 ln -s /usr/sbin/service %{buildroot}%{_sbindir}/rcwdmd
 %if %{with_fence_sanlockd}
-install -D -m 0755 %SOURCE5 
$RPM_BUILD_ROOT/usr/lib/systemd/systemd-fence_sanlockd
-install -D -m 0644 init.d/fence_sanlockd.service 
$RPM_BUILD_ROOT/%{_unitdir}/fence_sanlockd.service
+install -D -m 0755 %SOURCE5 %{buildroot}/usr/lib/systemd/systemd-fence_sanlockd
+install -D -m 0644 init.d/fence_sanlockd.service 
%{buildroot}/%{_unitdir}/fence_sanlockd.service
 ln -s /usr/sbin/service %{buildroot}%{_sbindir}/rcfence_sanlockd
 %endif
 %if %{with_sanlk_reset}
-install -D -m 0644 init.d/sanlk-resetd.service 
$RPM_BUILD_ROOT/%{_unitdir}/sanlk-resetd.service
+install -D -m 0644 init.d/sanlk-resetd.service 
%{buildroot}/%{_unitdir}/sanlk-resetd.service
 %endif
 %else
-install -D -m 755 %SOURCE1 $RPM_BUILD_ROOT/etc/init.d/sanlock
-ln -s /etc/init.d/sanlock $RPM_BUILD_ROOT%{_sbindir}/rcsanlock
-install -D -m 755 %SOURCE3 $RPM_BUILD_ROOT/etc/init.d/wdmd
-ln -s /etc/init.d/wdmd $RPM_BUILD_ROOT%{_sbindir}/rcwdmd
+install -D -m 755 %SOURCE1 %{buildroot}/etc/init.d/sanlock
+ln -s /etc/init.d/sanlock %{buildroot}%{_sbindir}/rcsanlock
+install -D -m 755 %SOURCE3 %{buildroot}/etc/init.d/wdmd
+ln -s /etc/init.d/wdmd %{buildroot}%{_sbindir}/rcwdmd
 %if %{with_fence_sanlockd}
-install -D -m 755 %SOURCE5 $RPM_BUILD_ROOT/%{_initddir}/fence_sanlockd
-ln -s /etc/init.d/fence_sanlockd $RPM_BUILD_ROOT%{_sbindir}/rcfence_sanlockd
+install -D -m 755 %SOURCE5 %{buildroot}/%{_initddir}/fence_sanlockd
+ln -s /etc/init.d/fence_sanlockd %{buildroot}%{_sbindir}/rcfence_sanlockd
 %endif
 %endif
 
 install -Dm 0644 src/logrotate.sanlock \
-       $RPM_BUILD_ROOT/etc/logrotate.d/sanlock
+       %{buildroot}/etc/logrotate.d/sanlock
 
-install -Dd -m 0755 $RPM_BUILD_ROOT/etc/wdmd.d
+install -Dd -m 0755 %{buildroot}/etc/wdmd.d
 
 %pre
 %{_bindir}/getent group sanlock > /dev/null || %{_sbindir}/groupadd \
@@ -224,19 +219,15 @@
 %service_add_pre sanlock.service
 %endif
 
-%if %{with_fence_sanlockd}
 %pre -n fence-sanlock
 %if %{with_systemd}
 %service_add_pre fence_sanlockd.service
 %endif
-%endif
 
-%if %{with_sanlk_reset}
 %pre -n sanlk-reset
 %if %{with_systemd}
 %service_add_pre sanlk-resetd.service
 %endif
-%endif
 
 %post
 %if %{with_systemd}
@@ -262,13 +253,11 @@
 #fi
 %endif
 
-%if %{with_sanlk_reset}
 %post -n sanlk-reset
 %if %{with_systemd}
 %service_add_post sanlk-resetd.service
 %endif
 %restart_on_update sanlk-resetd
-%endif
 
 %preun
 %if %{with_systemd}
@@ -278,23 +267,19 @@
 %stop_on_removal sanlock
 %endif
 
-%if %{with_fence_sanlockd}
 %preun -n fence-sanlock
 %if %{with_systemd}
 %service_del_preun fence_sanlockd.service
 %else
 %stop_on_removal fence_sanlockd
 %endif
-%endif
 
-%if %{with_sanlk_reset}
 %preun -n sanlk-reset
 %if %{with_systemd}
 %service_del_preun sanlk-resetd.service
 %else
 %stop_on_removal sanlk-resetd
 %endif
-%endif
 
 %postun
 %if %{with_systemd}
@@ -305,19 +290,15 @@
 
 %postun -n libsanlock1 -p /sbin/ldconfig
 
-%if %{with_fence_sanlockd}
 %postun -n fence-sanlock
 %if %{with_systemd}
 %service_del_postun fence_sanlockd.service
 %endif
-%endif
 
-%if %{with_sanlk_reset}
 %postun -n sanlk-reset
 %if %{with_systemd}
 %service_del_postun sanlk-resetd.service
 %endif
-%endif
 
 %files
 %defattr(-,root,root,-)


Reply via email to