Skip to content

apcupsd: Work on install#8092

Open
uni-dos wants to merge 1 commit intogetsolus:mainfrom
uni-dos:apcupsd-fix
Open

apcupsd: Work on install#8092
uni-dos wants to merge 1 commit intogetsolus:mainfrom
uni-dos:apcupsd-fix

Conversation

@uni-dos
Copy link
Copy Markdown
Contributor

@uni-dos uni-dos commented Mar 6, 2026

Summary

  • install license file
  • follow Fedora/Arch and install a .service file
  • Just work on install

Test Plan

Install and connect to ups and run apcaccess

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

Copy link
Copy Markdown
Member

@davidjharder davidjharder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@uni-dos
Copy link
Copy Markdown
Contributor Author

uni-dos commented Mar 23, 2026

I'm assuming the number in front of the .preset file is like priority?

@davidjharder
Copy link
Copy Markdown
Member

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

@uni-dos
Copy link
Copy Markdown
Contributor Author

uni-dos commented Mar 23, 2026

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants