-
Notifications
You must be signed in to change notification settings - Fork 175
build: update and shrink build images, migrate to clang 19 #3771
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
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
0ae7247
build: migrate clang 17 -> 19
morrisonlevi eb6cef6
ci: avoid interactive mode for docker login
morrisonlevi 7e770e9
ci: -u must come before --pasword-stdin
morrisonlevi d5713ad
build: drop LLVM components that need python, drop protobuf
morrisonlevi 1417cce
build: shrink centos-7 base image by ~2.4 GB
morrisonlevi f371b5f
build: bump PHP versions across CI images
morrisonlevi 33b0807
ci: drop ccmake, cpack, ctest
morrisonlevi 8cc639e
build: pin sqlsrv and grpc PECL versions for PHP compatibility
morrisonlevi 09b85da
build: patch PHP 8.5 to disable preserve_none under ASan
morrisonlevi 7da2707
build: silence make recipe output in bookworm PHP builds
morrisonlevi dac12df
build: bump CI images from bookworm-6/5 to bookworm-7
morrisonlevi 78895d3
Merge branch 'master' into levi/clang-17-to-19
morrisonlevi 86e867b
build: update clang/llvm references from 17 to 19
morrisonlevi f17823a
fix(centos-7): restore ctest, update clang/llvm/clang-tidy 17 → 19
morrisonlevi e5f9a20
ci: put bookworm next down to 7 in case I need to rebuild
morrisonlevi edb04ad
fix: update remaining llvm17/clang17 references to 19
morrisonlevi 27fc984
fix(appsec): find fuzzer runtime lib in Debian's linux/ layout
morrisonlevi 3ee2336
fix(appsec): pass __VA_ARGS__ through CONFIG to SYSCFG to avoid empty…
morrisonlevi dc150da
style(appsec): clang-format-19 helper sources
morrisonlevi 51f5f5b
ci: bump bookworm NEXT version to 8
morrisonlevi 59cff41
ci: revert CentOS 6 changes to avoid appearing that it's maintained
morrisonlevi a499e2b
fix(appsec): address clang-tidy 19 new checks
morrisonlevi 4461770
fix(appsec): separate void return from call expression in waf.cpp
morrisonlevi 1b9f92a
ci(appsec): revert clang-19 migration, keep clang-17, bump to bookworm-7
morrisonlevi c2459bd
ci(appsec): revert bookworm-7, keep bookworm-6
morrisonlevi e340959
ci: xfail ext/sockets/tests/gh21161.phpt on PHP 8.4 and 8.5
morrisonlevi 3d1009f
ci: suppress ASAN stack-use-after-return in sandbox observer bailout
morrisonlevi c5b4ec4
ci: suppress sandbox observer ASAN error in ZAI tests too
morrisonlevi 7b0a851
ci: use detect_stack_use_after_return=0 to suppress sandbox observer …
morrisonlevi fbfb73e
ci: remove accidentally committed scratch files
morrisonlevi f7dc60b
ci: revert unintended change
morrisonlevi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,2 @@ | ||
| BOOKWORM_CURRENT_VERSION=6 | ||
| BOOKWORM_NEXT_VERSION=6 | ||
| BOOKWORM_CURRENT_VERSION=7 | ||
| BOOKWORM_NEXT_VERSION=8 |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this to prevent tokens from leaking into the logs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something like that, something complained here, claude recommended this as more secure anyway and it seemed plausible enough while being harmless if wrong.