-
Notifications
You must be signed in to change notification settings - Fork 76
Fix a checksum exception about bazel-zig-cc. #335
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
Fix a checksum exception about bazel-zig-cc. #335
Conversation
lum1n0us
commented
Mar 27, 2023
6eab25a
to
ce8fefd
Compare
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.
LGTM - Verified this is correct https://github.com/proxy-wasm/proxy-wasm-cpp-host/actions/runs/4529975569/jobs/7978375729?pr=337
ce8fefd
to
d74210e
Compare
``` WARNING: Download from https://git.sr.ht/~motiejus/bazel-zig-cc/archive/v0.4.4.tar.gz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException Checksum was b3dd78971f0f831f62d84ae96656bdb0941b91b7c80097d69f5be213351cc6ff but wanted ad6384b4d16ebb3e5047df6548a195e598346da84e5f320250beb9198705ac81 ``` Signed-off-by: liang.he@intel.com <liang.he@intel.com>
d74210e
to
0d6bc8b
Compare
Any chance to update it to latest version while you're at it? Assuming it's a simple version bump. |
like v1.0.1? |
Yes. |
met an error likes:
Does it suggest that we should use |
Correct. You could also follow the official example and use:
and replace |
Could you fix DCO ( |
861366a
to
5ee8b49
Compare
5ee8b49
to
0d6bc8b
Compare
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.
Thanks!
Signed-off-by: liang.he@intel.com <liang.he@intel.com>