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 <osmith@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged