Conversation
omacom/omarchy#6897 adds the unit under default/systemd/user/ along with the watcher binary, an enable-user-units.sh entry, and a migration. Without an install line here, omarchy-settings would ship the unit only into the default/ template tree and never into the search path systemd actually reads. install/user/first-run/enable-user-units.sh enables its units in a single `systemctl --user enable --now` call, so a missing unit fails the whole call and first-run never completes -- the same breakage #6746 hit before 2aa5c50 added the omarchy-crash-watch.service line. Do not merge until _commit is bumped past the #6897 merge: the unit does not exist at the currently pinned 0534987, so this install would fail the build. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to omacom/omarchy#6897, which adds a PipeWire watcher that keeps the speaker and mic mute LEDs in step. That PR ships
default/systemd/user/omarchy-audio-mute-led-watch.service, the watcher binary, anenable-user-units.shentry, and a migration — but the unit only reaches/usr/lib/systemd/user/if this package installs it explicitly.Without this line,
omarchy-settingsships the unit into thedefault/template tree and nowhere systemd looks.install/user/first-run/enable-user-units.shenables its units in a singlesystemctl --user enable --nowcall, so the missing unit fails the whole call,omarchy-provision-first-runnever marksfirst-run-user, and first-run replays on every login.That is exactly what #6746 hit before 2aa5c50 added the
omarchy-crash-watch.serviceline, so this is the same one-line fix applied ahead of the breakage rather than after it.Both new
bin/scripts already ship — theomarchyPKGBUILD installsbin/*wholesale — so the unit is the only gap.Draft: ordering
source=pins_commit, currently0534987, wheredefault/systemd/user/omarchy-audio-mute-led-watch.servicedoes not exist. Merging this before_commitmoves past the #6897 merge would makeinstall -Dm644fail on a missing source and break theomarchy-settingsbuild.Merge this as part of the release that picks up #6897, or any time after
_commitis bumped past it. Keeping it in draft until then.🤖 Generated with Claude Code