tools: Fix incorrectly quoted *
in bash scripts
#39121
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.
Proposed changes:
In a few places, we do something like
[[ "$var" == "*/filename" ]]
, which checks for the variable being literally "*/filename" rather than interpreting the*
as a glob as intended.This removes the unnecessary quotes so it works right.
Other information:
Jetpack product discussion
None
Does this pull request change what data or activity we track or use?
No
Testing instructions:
tools/composer-update-monorepo.sh projects/plugins/beta/composer.lock
ortools/version-packages.sh projects/plugins/beta/composer.json
, see if it works right rather than complaining that the directory doesn't exist.