Skip to content

Install the idle-inhibit user unit for omarchy#8452 - #210

Open
itz4blitz wants to merge 2 commits into
omacom:masterfrom
itz4blitz:install-idle-inhibit-unit
Open

Install the idle-inhibit user unit for omarchy#8452#210
itz4blitz wants to merge 2 commits into
omacom:masterfrom
itz4blitz:install-idle-inhibit-unit

Conversation

@itz4blitz

@itz4blitz itz4blitz commented Aug 26, 2026

Copy link
Copy Markdown

mise ~/.config/mise/config.toml tools: gh@2.98.0

Summary

  • Lockstep with basecamp/omarchy#8452: omarchy-idle-inhibit.service lives under default/systemd/user/ and has to land in /usr/lib/systemd/user/ or systemd never finds it.
  • Adds the install -Dm644 line to both omarchy-settings and omarchy-settings-dev.
  • The daemon and probe binaries do not need PKGBUILD lines; the omarchy package already glob-installs bin/*.

Test plan

  • Confirm pkgbuilds/omarchy-settings/PKGBUILD and pkgbuilds/omarchy-settings-dev/PKGBUILD both install default/systemd/user/omarchy-idle-inhibit.service to /usr/lib/systemd/user/omarchy-idle-inhibit.service
  • Merge alongside omarchy#8452 so ExecStart=/usr/bin/omarchy-idle-inhibit-daemon has both the binary (omarchy package) and the unit (omarchy-settings)

omacom/omarchy#8452 ships omarchy-idle-inhibit.service under default/systemd/user/. The daemon binary is already picked up by the omarchy package's bin/ glob; without this install line systemd never finds the unit.

Co-authored-by: Cursor <cursoragent@cursor.com>
@greptile-apps

greptile-apps Bot commented Aug 26, 2026

Copy link
Copy Markdown

Greptile Summary

The PR conditionally installs the idle-inhibit systemd user unit from both stable and development settings packages, allowing packaging to succeed before the coordinated upstream unit lands.

  • Installs the unit under /usr/lib/systemd/user/ when present.
  • Preserves compatibility with current source revisions that do not yet contain the unit.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains; the existence checks resolve the previously reported package-build failure for source revisions that lack the unit.

Important Files Changed

Filename Overview
pkgbuilds/omarchy-settings/PKGBUILD The stable recipe now safely skips the absent upstream unit and will package it after the coordinated source pin advances.
pkgbuilds/omarchy-settings-dev/PKGBUILD The development recipe applies the same guarded installation while its tracked branch awaits the upstream unit.

Reviews (2): Last reviewed commit: "Skip the idle-inhibit unit when the sour..." | Re-trigger Greptile

Comment thread pkgbuilds/omarchy-settings/PKGBUILD Outdated
Stable is pinned to v4.0.1, which predates the unit, and settings-dev tracks quattro, which also lacks it until omacom/omarchy#8452 merges. An unconditional install aborts package() on those trees. Gate on the file so this recipe can land ahead of the upstream commit pin.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant