renovate fails to update sha256 hash for Bazel dependency #28650
Answered
by
rarkins
avdv
asked this question in
Request Help
-
What would you like help with?I think I found a bug How are you running Renovate?Mend Renovate hosted app on github.com If you're self-hosting Renovate, tell us which platform (GitHub, GitLab, etc) and which version of Renovate.No response Please tell us more about your question or problemrenovate created this PR: tweag/rules_sh#83 but somehow it failed to replace the old sha256 hash with the new one: diff --git a/bzlmod/extensions.bzl b/bzlmod/extensions.bzl
index b7fdd6f..4641127 100644
--- a/bzlmod/extensions.bzl
+++ b/bzlmod/extensions.bzl
@@ -64,7 +64,7 @@ def _sh_configure_impl(module_ctx):
http_file(
name = "rules_sh_shim_exe",
sha256 = "cb440b8a08a2095a59666a859b35aa5a1524b140b909ecc760f38f3baccf80e6",
- urls = ["https://github.com/ScoopInstaller/Shim/releases/download/v1.0.1/shim.exe"],
+ urls = ["https://github.com/ScoopInstaller/Shim/releases/download/v1.1.0/shim.exe"],
downloaded_file_path = "shim.exe",
executable = True,
)
diff --git a/sh/repositories.bzl b/sh/repositories.bzl
index 0ec6898..886891f 100644
--- a/sh/repositories.bzl
+++ b/sh/repositories.bzl
@@ -46,7 +46,7 @@ def rules_sh_dependencies():
http_file,
name = "rules_sh_shim_exe",
sha256 = "cb440b8a08a2095a59666a859b35aa5a1524b140b909ecc760f38f3baccf80e6",
- urls = ["https://github.com/ScoopInstaller/Shim/releases/download/v1.0.1/shim.exe"],
+ urls = ["https://github.com/ScoopInstaller/Shim/releases/download/v1.1.0/shim.exe"],
downloaded_file_path = "shim.exe",
executable = True,
)
Logs (if relevant)Logs
|
Beta Was this translation helpful? Give feedback.
Answered by
rarkins
Apr 29, 2024
Replies: 1 comment 3 replies
-
Looking at this:
It seems the current SHA isn't detected, which means SHA updating for http_file is not yet supported. It could be a small change to get it working, but I'm not sure. PRs welcome if you feel inclined to look into it yourself. If not, I can convert it into an issue once we have a minimal reproduction we can fork and debug. |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If Renovate didn't raise a PR then you might never know there's an update.
I recall that the bazel manager has some manual "massaging" of files and URLs for reasons like these. Here's an example I found:
renovate/lib/modules/manager/bazel/artifacts.ts
Lines 34 to 40 in 8749bc7
So you might be able to fix it that way. If not, just manually change the file and after that it…