Skip to content
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 tests.remoteBuilds_local_nix_2_18 #11832

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

edolstra
Copy link
Member

@edolstra edolstra commented Nov 7, 2024

Motivation

This was broken since a03bb44 because Nix 2.18 does not support broken $SHELL settings. So don't try a broken $SHELL on old Nix versions. (It's a mystery though why tests.remoteBuilds_local_nix_2_13 and tests.remoteBuilds_local_nix_2_3 didn't fail...)

https://hydra.nixos.org/build/277366807

Context

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

This was broken since a03bb44 because
Nix 2.18 does not support broken $SHELL settings. So don't try a
broken $SHELL on old Nix versions. (It's a mystery though why
tests.remoteBuilds_local_nix_2_13 and tests.remoteBuilds_local_nix_2_3
didn't fail...)

https://hydra.nixos.org/build/277366807
@edolstra edolstra requested a review from roberth November 7, 2024 19:49
@edolstra edolstra merged commit 723fdeb into NixOS:master Nov 7, 2024
11 checks passed
@edolstra edolstra deleted the fix-remote-build-test branch November 7, 2024 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants