Conversation
davidjharder
left a comment
There was a problem hiding this comment.
This PR seems to have fallen through the cracks, sorry.
These changes look fine, but do you want to take a stab at adding a preset file so the new service is also enabled on install?
Could also merge as is if you want, leave the preset thing for later
|
I'm assuming the number in front of the .preset file is like priority? |
|
Yeah, systemd reads the whole directory, then orders them by name, then there are shenanigans about what does and doesn't get set according to what was read last. Then upshot is we are setting priority 20 for all package preset as a general rule, and 20 is picked because there is "room" to have other files with lower or higher priority |
|
So I added the preset file but I think I need to install the .service file as well since it isn't provided. When I install the package with the changes apcupsd is disabled and preset is disabled. I installed keyd and it has both enabled. |
**Summary** - install license file - follow Fedora/Arch and install a .service file
Summary
Test Plan
Install and connect to ups and run apcaccess
Checklist