New comment by FollieHiyuki on void-packages repository https://github.com/void-linux/void-packages/pull/30195#issuecomment-818854386 Comment: I agree with moving the file using `post_install()`. I would rather patch the source to reflect the new config path: ```diff diff --git a/41_snapshots-btrfs b/41_snapshots-btrfs index 2a1f215..ca71206 100755 --- a/41_snapshots-btrfs +++ b/41_snapshots-btrfs @@ -41,7 +41,7 @@ prefix="/usr" exec_prefix="/usr" datarootdir="/usr/share" sysconfdir="/etc" -grub_btrfs_config="${sysconfdir}/default/grub-btrfs/config" +grub_btrfs_config="${sysconfdir}/conf.d/grub-btrfs" [[ -f "$grub_btrfs_config" ]] && . "$grub_btrfs_config" . "$datarootdir/grub/grub-mkconfig_lib" diff --git a/Makefile b/Makefile index 6ffe8c1..12c760d 100644 --- a/Makefile +++ b/Makefile @@ -14,7 +14,6 @@ install: exit 1; \ fi @install -Dm755 -t "$(DESTDIR)/etc/grub.d/" 41_snapshots-btrfs - @install -Dm644 -t "$(DESTDIR)/etc/default/grub-btrfs/" config @install -Dm644 -t "$(LIB_DIR)/systemd/system/" grub-btrfs.service @install -Dm644 -t "$(LIB_DIR)/systemd/system/" grub-btrfs.path @install -Dm644 -t "$(SHARE_DIR)/licenses/$(PKGNAME)/" LICENSE @@ -33,7 +32,6 @@ uninstall: fi @grub_dirname="$$(grep -oP '^[[:space:]]*GRUB_BTRFS_GRUB_DIRNAME=\K.*' "$(DESTDIR)/etc/default/grub-btrfs/config" | sed "s|\s*#.*||;s|(\s*\(.\+\)\s*)|\1|;s|['\"]||g")"; \ rm -f "$${grub_dirname:-/boot/grub}/grub-btrfs.cfg" - @rm -f "$(DESTDIR)/etc/default/grub-btrfs/config" @rm -f "$(DESTDIR)/etc/grub.d/41_snapshots-btrfs" @rm -f "$(LIB_DIR)/systemd/system/grub-btrfs.service" @rm -f "$(LIB_DIR)/systemd/system/grub-btrfs.path" ``` and use `/etc/conf.d/grub-btrfs` (instead of `grub-btrfs.cfg`) so that it aligns with our naming scheme (as for `snapper`) And maybe rename `grub-btrfs-runit` to `runit-grub-btrfs` for naming consistency (we have `runit-iptables` and `runit-kdump` for example)