Publish the packages omarchy depends on - #11
Merged
Merged
Conversation
omarchy declares omarchy-keyring and ttf-jetbrains-mono-nerd-basic as hard depends, build-packages.sh builds both, and verify() staged neither. They were produced and discarded on every run, and appear in no repository this one carries -- not omarchy-aarch64, not asahi-alarm, not core/extra/alarm, not the AUR. Machines have them only because the ISO installer put them there. Anything provisioned another way, or that loses one, cannot install or upgrade omarchy at all: the dependency is unsatisfiable and pacman refuses. The repo did not satisfy its own dependencies. Stage them alongside the pair, driven by what omarchy actually declares, so an omarchy that stops requiring one stops publishing it. They are arch=any and carry their own upstream versions, so neither the aarch64 assertion nor the RELEASE_VERSION assertion applies; what is checked is that they are the depends omarchy names, and that a named one which failed to build stops the release rather than publishing an omarchy nobody can install. Recorded in packages.json as skip:true against pkgbase omarchy, the way the pair already is, since update-omarchy-mac.yml owns them rather than the generic updater. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
malik-na
approved these changes
Sep 6, 2026
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
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.
omarchydeclaresomarchy-keyringandttf-jetbrains-mono-nerd-basicas hard depends.build-packages.shbuilds both.verify()staged neither — so they were produced and thrown away on every run.Neither exists in any repository this one carries. On a machine with
omarchy-aarch64,asahi-alarm,core,extra,alarmandaurconfigured:Machines have them only because the ISO installer put them there. Anything provisioned another way — or that loses one — cannot install or upgrade
omarchyat all, because the dependency is unsatisfiable and pacman refuses the transaction. The repo did not satisfy its own dependencies.What changed
Stage them alongside the pair, driven by what
omarchyactually declares, so anomarchythat stops requiring one stops publishing it — the list can't drift from the package.They're
arch=anyand carry their own upstream versions (omarchy-keyring 20251027-1,ttf-jetbrains-mono-nerd-basic 3.5.1-1), so neither theaarch64assertion nor theRELEASE_VERSIONassertion applies to them. What is enforced is that a declared dependency which failed to build stops the release, rather than publishing anomarchynobody can install.Recorded in
packages.jsonasskip: trueagainst pkgbaseomarchy, the way the pair already is, sinceupdate-omarchy-mac.ymlowns them rather than the generic updater.Verified
Ran
verify()as a subprocess against the real build output from thev4.0.2-2tag — all four stage:Two new self-tests: declared-and-present stages all four; declared-but-missing is rejected. 43/43 pass.
Scope
Pre-existing — these were depends of
4.0.1-2too, so this isn't a regression from the current release work. It surfaced while checking whether anyone could receive thev4.0.2-2update without something else they need.Worth landing before the
v4.0.2-2publish, so the repo becomes self-sufficient in the same release rather than shipping anotheromarchywhose depends can't be resolved.