-
-
Notifications
You must be signed in to change notification settings - Fork 17.4k
nix: fix cross-build failure by disabling s3-aws-auth #468205
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
Ericson2314
left a comment
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.
Can you send the same diff to upstream Nix?
5df771a to
c927d0e
Compare
There was already something like this upstream, so I tried to match it as closely as possible, while keeping the versioning support. |
|
|
| (lib.mesonEnable "seccomp-sandboxing" stdenv.hostPlatform.isLinux) | ||
| (lib.mesonBool "embedded-sandbox-shell" embeddedSandboxShell) | ||
| ] | ||
| ++ lib.optional (lib.versionAtLeast (lib.versions.majorMinor version) "2.33") ( |
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.
oh haha tricky, nice
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.
That option got renamed at the same time and doesn't exist before 2.33, so I thought this was best.
|
I think this became a problem with:
NixOS/nix@ffbc33f
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.