Arch Linux User Repository

Thanks for packaging this!

You're welcome :)

the license is, to be precise, GPL-3.0-only (see Cargo.toml)

Thanks, fixed. I think it was the only way to specify before ArchLinux switched to SPDX license identifiers some time ago.

Instead of mkdir; cp you can use install -Dm, see the Rust packaging guidelines

Also fixed.

It might be nice to package a sample hook (eg the one given by upstream in README.md that users can base themselves on (installed to /usr/share/doc/ -- I'm not sure I'd enable it by default, since eg I disagree with upstream's default of using notifications, having Exec=/usr/bin/reboot-arch-btw instead. Though you could always enable it by default and expect users to edit the package before installing)

I'm also the upstream maintainer. I'm thinking since quite some time how to best bundle the hooks in the package. See also https://github.com/rnestler/reboot-arch-btw/issues/33. I agree that not all users want the desktop notifications. So maybe just two example hooks in /usr/share/doc?

given it's probably doable with a simple awk script.

Regarding binary size / bloat: With 0.9.0 I could reduce the binary size by using libdbus instead of zbus. Due to dynamically linking to libdbus it reduced binary size to 2.3M from 4.2M.

Regarding simplicity: If you just need the hook your solutions looks perfect. You may still want to distinguish between reboot and session restart, but this can be done by just using two hooks.

I mainly use it interactively, whenever I suspect that something broke due to kernel upgrade or at the end of the workday to decide if I power-off or suspend the laptop. Some users have a loop running in the background to get regular notifications that they should reboot or restart the session.

I agree with @abdulhakeem, though -- I'm probably not going to use this one for long, given it's probably doable with a simple awk script. Thinking on this a bit more, it might just be as simple as installing eg

[Trigger]
Operation = Upgrade
Type = Package
Target = linux
Target = linux-lts
Target = systemd
Target = linux-firmware
Target = amd-ucode
Target = intel-ucode
Target = xorg-server
Target = xorg-xwayland

[Action]
Description = Check whether a reboot is required
When = PostTransaction
Exec = /usr/bin/echo 'System packages were upgraded, please reboot ASAP'

Thanks for packaging this! A couple notes: - The license is, to be precise, GPL-3.0-only (see Cargo.toml) - Instead of mkdir; cp you can use install -Dm, see the Rust packaging guidelines - It might be nice to package a sample hook (eg the one given by upstream in README.md that users can base themselves on (installed to /usr/share/doc/ -- I'm not sure I'd enable it by default, since eg I disagree with upstream's default of using notifications, having Exec=/usr/bin/reboot-arch-btw instead. Though you could always enable it by default and expect users to edit the package before installing)

Such a massive install for such a small functionality lol oh well