Re: [PATCH] spec: fix trigger already defined issue on systems without systemd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le 29/03/2016 16:48, Jean-Marc Liger a écrit :
Le 11/03/2016 15:18, Jean-Marc LIGER a écrit :
There is a trigger already defined issue when you try to rebuild libvirt >= 1.3.0 for el6 with copr and most probably koji.

Regards,
Jean-Marc

I've modified my patch after the nss update, someone could revue this patch before the next release ?

Regards,
Jean-Marc

diff -uri a/libvirt.spec.in b/libvirt.spec.in
--- a/libvirt.spec.in    2016-03-29 16:40:37.985927719 +0200
+++ b/libvirt.spec.in    2016-03-29 16:28:01.808864888 +0200
@@ -1783,16 +1783,6 @@
 fi
     %endif

-    %if %{with_systemd}
-    %else
-%triggerpostun daemon -- libvirt-daemon < 1.2.1
-if [ "$1" -ge "1" ]; then
-    /sbin/service virtlockd reload > /dev/null 2>&1 || :
-    /sbin/service virtlogd reload > /dev/null 2>&1 || :
-    /sbin/service libvirtd condrestart > /dev/null 2>&1
-fi
-    %endif
-
 # In upgrade scenario we must explicitly enable virtlockd/virtlogd
 # sockets, if libvirtd is already enabled and start them if
 # libvirtd is running, otherwise you'll get failures to start
@@ -1805,6 +1795,12 @@
         /bin/systemctl is-active libvirtd.service 1>/dev/null 2>&1 &&
             /bin/systemctl start virtlogd.socket || :
     %else
+    %triggerpostun daemon -- libvirt-daemon < 1.2.1
+    if [ "$1" -ge "1" ]; then
+        /sbin/service virtlockd reload > /dev/null 2>&1 || :
+        /sbin/service virtlogd reload > /dev/null 2>&1 || :
+        /sbin/service libvirtd condrestart > /dev/null 2>&1
+    fi
         /sbin/chkconfig libvirtd 1>/dev/null 2>&1 &&
             /sbin/chkconfig virtlogd on || :
         /sbin/service libvirtd status 1>/dev/null 2>&1 &&

Maybe this is the wrong way to fix this problem. But it exists for non systemd builds, where directives %triggerpostun daemon -- libvirt-daemon < 1.2.1 and %triggerpostun daemon -- libvirt-daemon < 1.3.0 are examined sequentialy, which is causing a trigger issue.

https://copr.fedorainfracloud.org/coprs/jmliger/tests-upstream/build/172145/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]