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

nixos/nix-serve: fix module compatibility with unflaked Nix #269280

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

tazjin
Copy link
Member

@tazjin tazjin commented Nov 22, 2023

The option extra-allowed-users was introduced in Nix 2.4, and fails config validation on Nix 2.3.

Relates to NixOS/nix#9412

Breakage discovered in https://cl.tvl.fyi/c/depot/+/10083, commit that introduced the bug was 44cf480

The option `extra-allowed-users` was introduced in Nix 2.4, and fails
config validation on Nix 2.3.
@flokli
Copy link
Contributor

flokli commented Nov 22, 2023

Successfully ran nixosTests.nix-serve with nix.package = pkgs.nix_2_3; added to the VM config. Also, validated the test still is the same when Nix is left to the default.

@flokli flokli merged commit ab21317 into NixOS:master Nov 22, 2023
7 of 8 checks passed
@tazjin tazjin deleted the nix-serve-unflaked branch November 22, 2023 21:44
Copy link
Contributor

Successfully created backport PR for release-23.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants