CA-399643 Use full paths when stopping fairlock on upgrade

Amend the fairlock upgrade scriptlet so that it uses full paths. This prevents it failing unexpectedly. Also, change it from a %posttrans scriptlet to a %post scriptlet so that the new version will have an effect immediately any new version is available. Signed-off-by: Tim Smith <[email protected]>
xapi-project · Sep 26, 2024 · 24de268 · 24de268
1 parent 130615c
commit 24de268
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/mk/sm.spec.in b/mk/sm.spec.in
@@ -237,13 +237,16 @@ Manager and some other packages
 %{_unitdir}/[email protected]
 %{_libexecdir}/fairlock
 
-%posttrans fairlock
+%post fairlock
 ## On upgrade, shut down existing lock services so new ones will
 ## be started. There should be no locks held during upgrade operations
 ## so this is safe.
 if [ $1 -gt 1 ];
 then
-    systemctl stop $(systemctl list-units fairlock@* --all --no-legend | cut -d' ' -f1)
+    /usr/bin/systemctl list-units fairlock@* --all --no-legend | /usr/bin/cut -d' ' -f1 | while read service;
+    do
+        /usr/bin/systemctl stop "$service"
+    done
 fi
 
 %changelog