laforge submitted this change.

View Change


Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
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(-)

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 change 36648. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I89802f926bfccc0f7b4bb1ff64115b232b1db022
Gerrit-Change-Number: 36648
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged