laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-sgsn/+/36648?usp=email )
Change subject: contrib/osmo-sgsn.spec: fix build for almalinux:8
......................................................................
contrib/osmo-sgsn.spec: fix build for almalinux:8
Add the missing "%if 0%{?suse_version}" around %service_del/add
commands, as these are only available on opensuse.
Fix for:
error: line 106: Too many names: %preun -n osmo-gtphub %service_del_preun
osmo-gtphub.service
Fixes: a07e6d9c (".deb/.rpm: add osmocom user during package install")
Change-Id: I89802f926bfccc0f7b4bb1ff64115b232b1db022
---
M contrib/osmo-sgsn.spec.in
1 file changed, 18 insertions(+), 0 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/contrib/osmo-sgsn.spec.in b/contrib/osmo-sgsn.spec.in
index d1382cb..031cd14 100644
--- a/contrib/osmo-sgsn.spec.in
+++ b/contrib/osmo-sgsn.spec.in
@@ -100,10 +100,12 @@
chown root:osmocom /etc/osmocom
chmod 2775 /etc/osmocom
+%if 0%{?suse_version}
%preun -n osmo-gtphub %service_del_preun osmo-gtphub.service
%postun -n osmo-gtphub %service_del_postun osmo-gtphub.service
%pre -n osmo-gtphub %service_add_pre osmo-gtphub.service
%post -n osmo-gtphub %service_add_post osmo-gtphub.service
+%endif
%check
make %{?_smp_mflags} check || (find . -name testsuite.log -exec cat {} +)
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/36648?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I89802f926bfccc0f7b4bb1ff64115b232b1db022
Gerrit-Change-Number: 36648
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
