-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
buildBazelPackage: fix difference between linux and darwin deps #224917
Conversation
@ofborg build envoy |
@ofborg build verible |
@ofborg build bazel-watcher |
@ofborg build python3Packages.tensorflow |
@ofborg build python3Packages.jaxlib-build |
@ofborg build python3Packages.tensorflow-probability |
80feb1e
to
f8ee061
Compare
@ofborg build envoy |
@@ -81,7 +81,7 @@ buildBazelPackage rec { | |||
fetchAttrs = { | |||
sha256 = { | |||
x86_64-linux = "sha256-H2s8sTbmKF+yRfSzLsZAT2ckFuunFwh/FMSKj+GYyPM="; | |||
aarch64-linux = "sha256-R9jzy/dpdCcGgT9yq59Wo/IN/bVo6fxnVPGhLMZ9fbM="; | |||
aarch64-linux = "sha256-1/z7sZYMiuB4Re2itDZydsFVEel2NOYmi6vRmBGVO/4="; |
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.
I don't know why this failed, but it did here: https://logs.nix.ci/?key=nixos/nixpkgs.224917&attempt_id=1a72d82c-2616-408d-b007-23f91f7a0e00
The changes in this PR wouldn't affect this hash at all, but it might have been affected by a past change and we never noticed because it was getting it from the cache.
@@ -398,7 +398,7 @@ let | |||
else "sha256-QgOaUaq0V5HG9BOv9nEw8OTSlzINNFvbnyP8Vx+r9Xw="; | |||
aarch64-linux = "sha256-zjnRtTG1j9cZTbP0Xnk2o/zWTNsP8T0n4Ai8IiAT3PE="; | |||
x86_64-darwin = "sha256-RBLox9rzBKcZMm4NwnT7vQ/EjapWQJkqxuQ0LIdaM1E="; |
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.
This hash has changed as well but since this is currently broken and can't be built on ofborg I have no way of updating it. Whoever will fix this will need to update it.
@@ -239,7 +239,7 @@ let | |||
x86_64-linux = "sha256-A0A18kxgGNGHNQ67ZPUzh3Yq2LEcRV7CqR9EfP80NQk="; | |||
aarch64-linux = "sha256-mU2jzuDu89jVmaG/M5bA3jSd7n7lDi+h8sdhs1z8p1A="; | |||
x86_64-darwin = "sha256-9nNTpetvjyipD/l8vKlregl1j/OnZKAcOCoZQeRBvts="; |
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.
This hash has changed as well but since this is currently broken and can't be built on ofborg I have no way of updating it. Whoever will fix this will need to update it.
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.
It looks good to me 👍
Description of changes
When tarring the broken symlinks the mode in the entries depends on the mode of the symlink. On darwin the symlinks are 755, but on linux they are 777, and their mode cannot be changed. This removes that source of difference between linux and darwin in deps archives.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)