Skip to content
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

Add [Install] section to aux systemd units #131

Open
wants to merge 1 commit into
base: fedora-39
Choose a base branch
from

Conversation

LorbusChris
Copy link
Contributor

@LorbusChris LorbusChris commented Aug 9, 2023

Currently in Fedora, these services are statically enabled by symlinks, with no other way to disable them than to manually delete those symlinks:

https://src.fedoraproject.org/rpms/grub2/blob/dc5c4e3f52ab2a829168e81bd1e9eafced9b7c09/f/grub2.spec#_274-284

This is problematic in Fedora IoT, where grub-boot-success.timer is not supposed to be enabled.

This change adds [Install] sections to all systemd units that are currently enabled statically, so that they can be enabled dynamically via presets or manually via systemctl instead.

See

Currently in Fedora, these services are statically enabled by symlinks,
with no other way to disable them than to manually delete those symlinks.
This is problematic in Fedora IoT, where grub-boot-success.timer is
not supposed to be enabled.

This change adds `[Install]` sections to all systemd units that are
currently enabled statically, so that they can be enabled dynamically
via presets or manually instead.
@LorbusChris
Copy link
Contributor Author

The preset addition has already been merged:
https://src.fedoraproject.org/rpms/fedora-release/pull-request/274

Once this one here is in, the follow up is https://src.fedoraproject.org/rpms/grub2/pull-request/26

@LorbusChris
Copy link
Contributor Author

cc @jwrdegoede @vathpela

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