- bsc#1210303: Stop using the obsolete internal %_restart_on_update

macro and drop support for sysv init to simplify the scriptlets.

OBS-URL: https://build.opensuse.org/package/show/server:database:postgresql/postgresql15?expand=0&rev=23

postgresql15.changes

@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue Apr 18 09:05:09 UTC 2023 - Reinhard Max <max@suse.com>
+
+- bsc#1210303: Stop using the obsolete internal %_restart_on_update
+  macro and drop support for sysv init to simplify the scriptlets.
+
 -------------------------------------------------------------------
 Tue Apr  4 10:57:41 UTC 2023 - Fabian Vogt <fvogt@suse.com>
 

postgresql15.spec

@@ -95,13 +95,6 @@ BuildRequires:  pkgconfig(libzstd)
 BuildRequires:  %libecpg
 BuildRequires:  %libpq
 %endif
-%if 0%{?suse_version} >= 1300
-%bcond_without  systemd
-%bcond_without  systemd_notify
-%else
-%bcond_with     systemd
-%bcond_with     systemd_notify
-%endif
 
 %if 0%{?suse_version} >= 1500 && %pgmajor >= 11
 %ifarch riscv64
@@ -148,10 +141,8 @@ BuildRequires:  openldap2-devel
 BuildRequires:  openssl-devel
 BuildRequires:  pkg-config
 BuildRequires:  pkgconfig(krb5)
-%if %{with systemd_notify}
 BuildRequires:  pkgconfig(libsystemd)
 BuildRequires:  pkgconfig(systemd)
-%endif
 #!BuildIgnore:  postgresql-implementation
 #!BuildIgnore:  postgresql-server-implementation
 #!BuildIgnore:  postgresql-devel-noarch
@@ -547,9 +538,7 @@ PACKAGE_TARNAME=%pgname %configure \
 %if %{with libzstd}
         --with-zstd \
 %endif
-%if %{with systemd_notify}
         --with-systemd \
-%endif
 %if %{with selinux}
         --with-selinux \
 %endif
@@ -818,45 +807,28 @@ awk -v P=%buildroot '/^(%lang|[^%])/{print P $NF}' libpq.files libecpg.files | x
 
 %preun server
 # Stop only when we are uninstalling the currently running version
-test -n "$FIRST_ARG" || FIRST_ARG="$1"
+test -x /usr/bin/systemctl &&
-if [ "$FIRST_ARG" -eq 0 ]; then
+  MAINPID=$(/usr/bin/systemctl show postgresql.service --property=MainPID --value) ||:
-  %if %{with systemd}
+if test -n "$MAINPID" && test "$MAINPID" -ne 0; then
-    %define stop systemctl stop postgresql.service
+  BIN=$(readlink -n /proc/$MAINPID/exe)
-    eval $(systemctl show postgresql.service --property=MainPID)
+  DIR=$(dirname ${BIN% *})
-  %else
+  if test "$DIR" = "%pgbindir" -o "$DIR" = "%_bindir"; then
-    %define stop /sbin/init.d postgresql stop
+      %service_del_preun postgresql.service
-    MainPID=$(pidof -s postgres) || :
-  %endif
-  if test -n "$MainPID" && test "$MainPID" -ne 0; then
-    BIN=$(readlink -n /proc/$MainPID/exe)
-    DIR=$(dirname ${BIN% *})
-    if test "$DIR" = "%pgbindir" -o "$DIR" = "%_bindir"; then
-        %stop
-    fi
   fi
 fi
 
 %postun server
 /usr/share/postgresql/install-alternatives %pgmajor
 # Restart only when we are updating the currently running version
-# or from the old packaging scheme
+test -x /usr/bin/systemctl &&
-test -n "$FIRST_ARG" || FIRST_ARG="$1"
+  MAINPID=$(/usr/bin/systemctl show postgresql.service --property=MainPID --value) ||:
-if [ "$FIRST_ARG" -ge 1 ]; then
+if test -n "$MAINPID" && test "$MAINPID" -ne 0 &&
-  %if %{with systemd}
+  readlink -n /proc/$MAINPID/exe | grep -Fq " (deleted)"
-    %define restart %_restart_on_update postgresql.service
+then
-    eval $(systemctl show postgresql --property=MainPID)
+  BIN=$(readlink -n /proc/$MAINPID/exe)
-  %else
+  DIR=$(dirname ${BIN% *})
-    %define restart /sbin/init.d postgresql restart
+  if test "$DIR" = "%pgbindir" -o "$DIR" = "%_bindir"; then
-    MainPID=$(pidof -s postgres) || :
+      %service_del_postun postgresql.service
-  %endif
-  if test -n "$MainPID" && test "$MainPID" -ne 0 &&
-    readlink -n /proc/$MainPID/exe | grep -Fq " (deleted)"
-  then
-    BIN=$(readlink -n /proc/$MainPID/exe)
-    DIR=$(dirname ${BIN% *})
-    if test "$DIR" = "%pgbindir" -o "$DIR" = "%_bindir"; then
-        %restart
-    fi
   fi
 fi
 
@@ -938,6 +910,7 @@ fi
 %endif
 
 %files llvmjit-devel
+%defattr(-,root,root)
 %doc README
 
 %files pltcl -f pltcl.lang
@@ -965,12 +938,13 @@ fi
 
 %if %buildlibs && %mini
 %files -n %pgname-%devel -f devel.files -f libpq.files -f libecpg.files
+%defattr(-,root,root)
 %else
 
 %files -n %pgname-%devel -f devel.files
+%defattr(-,root,root)
 %endif
 
-%defattr(-,root,root)
 %dir %pgbasedir
 %dir %pgbindir
 %ghost %_bindir/ecpg