Repository navigation
docs: fix compat-update binary + live-verify recipes - #19
Merged
Merged
Conversation
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.
Follow-up to the 18.12.3 run: the command file is what future runs execute verbatim, and two of its recipes are provably wrong on this machine.
Step 2 — embedded-changelog source
readlink -f "$(which omp)"resolves the stale Homebrew Cellar 18.8.6 in a login shell (it has no 18.12.x window at all), and macOSreadlink -fdoes not exist. Replaced with a content-resolved probe overtype -a ompcandidates plus the Tern version store, picking the first bundle that actually carries## [TARGET].Two traps encoded, both hit during this fix:
grep -qis unusable underset -o pipefailhere: it closes the pipe,stringsdies of SIGPIPE, and theifsees 141 — a real match is silently discarded. The recipe usesgrep -cand tests the count.for b in $(...)splits Tern's space-containing paths;while IFS= read -rdoes not.Verified by executing the block verbatim:
BIN=/Users/rector/Library/Application Support/Tern/omp/bin/ompand the 18.12.3 window prints.Step 4 — live tmux verify
A brand-new
tmux -L <sock>server is not fresh when~/.tmux.confloads tmux-resurrect/continuum:@continuum-restore 'on'+@resurrect-capture-pane-contents 'on'restore the last saved layout with pane snapshots, socapture-panereturns a days-old widget render (observed:omp 18.6.1 · sl 0.6.3, dated 07 Oct) that reads as a broken install. Recipe now starts the server with-f /dev/null, launches the content-resolved$BIN(notcommand -v omp), captures with an explicit-t compat:0.0, and notes the inert leftover socket file.Docs only; no code, no version bump.