Update executables.txt in brew update when needed
#20739
Merged
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.
Now that
which-formulalives in this repo, we should fetchexecutables.txtonbrew updateinvocations. When a user runsbrew which-formulafor the first time, a setting is saved indicating thatexecutables.txtshould be downloaded on futurebrew updatecalls. If a user does not runbrew which-formula, the file will not be fetched.To make this easier, I've extracted the API file fetch logic to a helper function in
update.sh. You can view that change in isolation in1f042a3(#20739).I've also added
--skip-updateto the shell handlers that callwhich-formulaso no new files will be downloaded if a user types the wrong command in their shell.