-
Notifications
You must be signed in to change notification settings - Fork 81
chore: bump Bazel from 6.5.0 to 7.7.1 #475
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
Conversation
896c6af to
b7f9cdc
Compare
|
Hi @mmorel-35, do you mind if I push a few commits onto your branch? I think I might be able to resolve some of the build issues. |
|
Take all you need. I won’t be able to do anything in the next weeks . Feel free to proceed . |
41bccb2 to
ac64548
Compare
|
Great work getting everything but the macos CI running! This is a huge improvement to the dependency management of proxy-wasm. I will look into the llvm-ar issue affecting V8 on macos CI. We may be able to fix the s390x CI by reverting the change I made to use python 3.11. |
…figuration Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
…optimization flags Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
…--copt flags Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
… Bazel bootstrap on s390x Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Major Changes
Bazel Upgrade
Linting System Modernization
//tools/lint:linters.bzl%clang_tidy)Bzlmod Preparation (from native-bind and platforms branches)
native.bind()calls removed from bazel/repositories.bzl. BUILD file now references dependencies directly using canonical labels (e.g.,@v8//:wee8instead of//external:wee8)Dependency Updates
New Dependencies for Modern Linting
WORKSPACE Configuration
Standard load order maintained:
Infrastructure Changes
tools/lint/BUILDwith hermetic clang-tidy binary configurationtools/lint/linters.bzlwith clang_tidy aspect configurationnative.bind()calls and updated BUILD file to use canonical dependency labelsBuild Validation
✅ Bazel Upgrade: Successfully upgraded to 7.7.1
✅ Linting Infrastructure: Modern aspect_rules_lint integration configured
✅ Main Library Build:
bazel build --config clang-tidy //:libcompletes successfully✅ C++ Tests: Build successfully with clang-tidy
✅ Bzlmod Preparation: native.bind() removed, direct dependency references working
Fixes #403
Fixes #429
Fixes #463
Fixes #466
Fixes #467
Fixes #468