Conversation
Update Dockerfile ARG LLVM_VER (both stages) and CI.yml env from 20 to 21 so the LLVM build pulls release/21.x. Existing >= 20 conditionals already cover the projects/runtimes split and the flang-new -> flang rename, which became hard requirements in LLVM 21.
…ang-rt LLVM 21 reorganized Flang: - flang/lib/Common → flang/lib/Support (lib FortranCommon → FortranSupport) - flang/runtime → flang-rt subproject (built via LLVM_ENABLE_RUNTIMES) Add LLVM_VER >= 21 branch to LLVM_RUNTIMES (adds flang-rt) and to FLANG_TARGETS (uses install-FortranSupport, drops install-FortranRuntime since install-runtimes aggregate now handles flang-rt). Backwards compatible: LLVM 20 path unchanged. Verified all 27 explicit ninja targets resolve in LLVM 21 via configure-only probe build (cmake -GNinja, then ninja -t query for each target).
…din JSON
Claude Code no longer exports CLAUDE_FILE_PATH; PostToolUse hooks now receive
input as JSON on stdin. All 7 hooks were silently no-op'ing (or erroring on
'unbound variable' under set -u) until tool-input was extracted via jq.
- Read file_path from .tool_input.file_path on stdin
- Add 'set -euo pipefail' to two hooks that lacked it
- Fix latent regex in check-workflow-expressions.sh to accept YAML list
dash prefix ('- run: |'), which had been masked by the no-op behavior
Tested each hook against pass/fail fixtures in .tmp/hook-test/ and confirmed
correct exit codes plus violation output. Edge cases (no path, empty JSON)
exit 0 cleanly.
Ref: https://code.claude.com/docs/en/hooks.md
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
1 participant
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.
Update Dockerfile ARG LLVM_VER (both stages) and CI.yml env from 20 to 21 so the LLVM build pulls release/21.x. Existing >= 20 conditionals already cover the projects/runtimes split and the flang-new -> flang rename, which became hard requirements in LLVM 21.