-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add protoc v26.0 and protobuf-ts v2.9.4 #1116
Conversation
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.
Initial commit contains the initial code from fetcher. 305a866 are my deltas on top.
@@ -1,3 +1,3 @@ | |||
FROM gcr.io/bazel-public/bazel:7.1.0 | |||
FROM gcr.io/bazel-public/bazel:7.0.2 |
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 the new bazel is doing is breaking protoc builds. I tested some locally and couldn't find a solution so reverting this back for now.
Build failures look like:
$ bazel build '//plugins:protoc-gen-cpp.stripped'
ERROR: Failed to initialize sandbox: /home/build/.cache/bazel/_bazel_build/433461e849f916304f45cf88ae617024/sandbox/_moved_trash_dir -> /home/build/.cache/bazel/_bazel_build/433461e849f916304f45cf88ae617024/sandbox/stale-trash-0 (Invalid cross-device link)
INFO: Elapsed time: 0.198s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
registry: | ||
maven: | ||
deps: | ||
- com.google.protobuf:protobuf-java:4.26.0 |
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.
Major version of dependencies bumped for Java.
kotlin: | ||
version: 1.8.22 | ||
deps: | ||
- com.google.protobuf:protobuf-kotlin:4.26.0 |
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.
Major version bump for kotlin too.
No description provided.