Skip to content

nixos services.xserver.displayManager.session: drop type#164895

Merged
vcunat merged 1 commit intoNixOS:masterfrom
vcunat:p/dm-session-typing
Mar 19, 2022
Merged

nixos services.xserver.displayManager.session: drop type#164895
vcunat merged 1 commit intoNixOS:masterfrom
vcunat:p/dm-session-typing

Conversation

@vcunat
Copy link
Member

@vcunat vcunat commented Mar 19, 2022

For now at least. I expect someone will find a working type later.
It's incorrect and was causing bad issues. Example test case:
nix-instantiate nixos/release.nix -A tests.xfce.x86_64-linux --dry-run

This is a partial revert of commit b2d803c from PR #162271.

@vcunat vcunat added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 1.severity: channel blocker Blocks a channel labels Mar 19, 2022
@github-actions github-actions bot added the 8.has: module (update) This PR changes an existing module in `nixos/` label Mar 19, 2022
@vcunat
Copy link
Member Author

vcunat commented Mar 19, 2022

OMG no... a type-less option is not allowed now?

For now at least.  I expect someone will find a working type later.
It's incorrect and was causing bad issues.  Example test case:
nix-instantiate nixos/release.nix -A tests.xfce.x86_64-linux --dry-run

This is a partial revert of commit b2d803c from PR NixOS#162271.
@vcunat vcunat force-pushed the p/dm-session-typing branch from 8dbf646 to fd609f9 Compare March 19, 2022 20:11
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Mar 19, 2022
@vcunat vcunat merged commit 02f6f9c into NixOS:master Mar 19, 2022
@vcunat vcunat deleted the p/dm-session-typing branch March 19, 2022 20:51
@vcunat
Copy link
Member Author

vcunat commented Mar 19, 2022

I was impatient with this.

cole-h referenced this pull request Mar 19, 2022
Co-Authored-By: Janne Heß <janne@hess.ooo>
@infinisil
Copy link
Member

OMG no... a type-less option is not allowed now?

Glad it's working, that's the goal of #162271 :)

This reversal looks good for now. While types.attrs isn't very good and will be deprecated in the future, we can defer the decision for the correct type until then.

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

Labels

1.severity: channel blocker Blocks a channel 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants