Overview

Request 678270 accepted

- Introduce %service_del_postun_without_restart
This is another step to make the SUSE rpm macros closer to their
upstream counterparts.
Introduce %service_del_postun_without_restart which should help in
getting rid of '-f' and '-n' switches of %service_del_postun.


Thomas Blume's avatar

reviewed OK


Marcello Barnaba's avatar

Hello,

this is causing the following error on many packages I rebuild for SuSE 42.X:

[ 47s] + exec rpmbuild -ba --define '_srcdefattr (-,root,root)' --nosignature --define 'disturl obs://build.opensuse.org/home:vjt:ifad/openSUSE_Leap_42.3/a46ed75bc886b8c0918b372b5c94af98-redis' /home/abuild/rpmbuild/SOURCES/redis.spec [ 47s] error: Too many levels of recursion in macro expansion. It is likely caused by recursive macro declaration. [ 47s] error: line 141: [ 47s] test -n "$FIRST_ARG" || FIRST_ARG="$1"
[ 47s] [ 47s] test -n "$FIRST_ARG" || FIRST_ARG="$1"
[ 47s] if [ "$FIRST_ARG" -eq 0 ]; then
[ 47s] # Package removal
[ 47s] for service in

Referencing the newly introduced macro. As far as I can tell it's due to how it is invoked, on line 188 of current macros.systemd:

%service_del_postun_without_restart %{?*} \

Thank you for looking into this,


Franck Bui's avatar
author target maintainer

Since this has been submitted to Base:System, I'm not sure to understand how you ended up with this new version of systemd-rpm-macros.

Can you provide more details on the failing build or a link that shows the failure ?

Request History
Franck Bui's avatar

fbui created request

- Introduce %service_del_postun_without_restart
This is another step to make the SUSE rpm macros closer to their
upstream counterparts.
Introduce %service_del_postun_without_restart which should help in
getting rid of '-f' and '-n' switches of %service_del_postun.


Thomas Blume's avatar

tsaupe accepted request

Ok

openSUSE Build Service is sponsored by