https://bugzilla.redhat.com/show_bug.cgi?id=2120119 --- Comment #29 from Alessandro Astone <ales.astone@xxxxxxxxx> --- (In reply to Neal Gompa from comment #28) > > - systemd_post is invoked in %post, systemd_preun in %preun, and > > systemd_postun in %postun for Systemd service files. > > Note: Systemd service file(s) in waydroid > > See: https://docs.fedoraproject.org/en-US/packaging- > > guidelines/Scriptlets/#_scriptlets > > This is because "%systemd_post" is run in %posttrans rather than %post. I > don't see a reason that this can't be moved to %post. My memory is not super fresh but this is what i remember: The reasoning was that I needed the systemd service to (re)start *after* the selinux changes take place. Now, I see that my %selinux_relabel_post is also in %posttrans which I probably took from the first code snippet here https://fedoraproject.org/wiki/SELinux/IndependentPolicy#The_%post_Section , so I put the %systemd_post in %posttrans as well because %post would be too soon. However I see that the second code snippet in the same link above prefers putting them both in %post instead. As well as %systemd_postun_with_restart in `%post selinux` for when selinux is only installed after the main package. I'll verify whether it keeps the right behaviour and update the spec. -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2120119 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
