Skip to content

packagin/postinst: Allow systemd reload to fail #118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Image for: Conversation
Copy link
Collaborator

agx commented May 9, 2025

During boostrapping of images there's no systemd yet so allow it to fail. This matches the behavior of the postrm snippets that dh_systemd creates. Since we don't care about policy-rc.d use systemctl directly.

This hopefully unbreaks the phosh nightly builds

https://salsa.debian.org/agx/phosh-recipes/-/jobs/7558422#L5481

(we didn't notice earlier as meta-phosh still had a dependency on phog and pulled that in, I've removed phog from the archive now and so we hit this)

During boostrapping of images there's no systemd yet so allow it to
fail. This matches the behavior of the postrm snippets that dh_systemd
creates. Since we don't care about policy-rc.d use systemctl directly.

This hopefully unbreaks the phosh nightly builds

https://salsa.debian.org/agx/phosh-recipes/-/jobs/7558422#L5481

Signed-off-by: Guido Günther <agx@sigxcpu.org>
Copy link

github-actions bot commented May 9, 2025

The CI build recorded and generated some videos

Demo video (shown on README and release notes)

accent-colours

emergency-calls

first-run

simple-flow

trivial-flow

Copy link
Collaborator Author

agx commented May 17, 2025

With this patch applied, the build succeeds: https://salsa.debian.org/agx/phosh-recipes/-/pipelines/866574

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant