Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix string handling in jlchecksum #56720

Merged
merged 1 commit into from
Dec 1, 2024
Merged

Fix string handling in jlchecksum #56720

merged 1 commit into from
Dec 1, 2024

Conversation

Zentrik
Copy link
Member

@Zentrik Zentrik commented Nov 30, 2024

A TAGGED_RELEASE_BANNER with spaces such as Official https://julialang.org release produces the error /cache/build/builder-amdci4-5/julialang/julia-master/deps/tools/jlchecksum: 66: [: Official: unexpected operator.

A ` TAGGED_RELEASE_BANNER` with spaces such as `Official https://julialang.org release` produces the error `/cache/build/builder-amdci4-5/julialang/julia-master/deps/tools/jlchecksum: 66: [: Official: unexpected operator`.
@giordano giordano added the building Build system, or building Julia or its dependencies label Nov 30, 2024
@giordano giordano merged commit ef32806 into master Dec 1, 2024
9 checks passed
@giordano giordano deleted the Zentrik-patch-1 branch December 1, 2024 04:35
stevengj pushed a commit that referenced this pull request Jan 2, 2025
A `TAGGED_RELEASE_BANNER` with spaces such as `Official
https://julialang.org release` produces the error
`/cache/build/builder-amdci4-5/julialang/julia-master/deps/tools/jlchecksum:
66: [: Official: unexpected operator`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants