Fix detect_clang_version version extraction #13335
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.
The current pattern matching
/clang-(\d+\.\d+\.\d+(\.\d+)?)/, 1
in thedetect_clang_version
method fails to fully match the new clang version string"1316.0.21.2.5"
introduced with CLT 13.4, which lead to a false report of outdated CLT.With this rewritten regex
/clang-(\d+(\.\d+)+)/, 1
, we have a more robust pattern matching enabled.brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?