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

GDM fixes #29697

Merged
merged 3 commits into from Sep 27, 2017
Merged

GDM fixes #29697

merged 3 commits into from Sep 27, 2017

Conversation

zimbatm
Copy link
Member

@zimbatm zimbatm commented Sep 23, 2017

Motivation for this change

This is a rework of #25621

  • disable the GDM wayland mode by default
  • fixes the nvidia permission issues
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@zimbatm zimbatm changed the base branch from master to release-17.09 September 23, 2017 14:17
@FRidh FRidh removed their request for review September 23, 2017 14:19
@peti peti removed their request for review September 23, 2017 14:19
@zimbatm zimbatm force-pushed the gdm-on-nvidia branch 6 times, most recently from 333dd65 to 39f39b4 Compare September 23, 2017 16:19
@zimbatm
Copy link
Member Author

zimbatm commented Sep 24, 2017

we are still trying to fix the gnome3-gdm tests

@teh
Copy link
Contributor

teh commented Sep 24, 2017

@zimbatm you might want to be explicit about true and false because toString behaves as follows:

nix-repl> toString true
"1"

i.e.

[daemon]
-      WaylandEnable=${toString cfg.gdm.wayland}
+      WaylandEnable=${if cfg.gdm.wayland then "true" else "false"}
       ${optionalString cfg.gdm.autoLogin.enable (

@copumpkin
Copy link
Member

Any idea why it seems to think I'm an owner here?

@zimbatm
Copy link
Member Author

zimbatm commented Sep 25, 2017

I don't know, when I created the issue, a bunch of people were automatically added as reviewers

@dezgeg
Copy link
Contributor

dezgeg commented Sep 26, 2017

Probably because the base branch was incorrect the first time, GitHub though you're proposing to submit every single commit from 17.09 in master.

@fpletz fpletz added this to the 17.09 milestone Sep 26, 2017
@fpletz fpletz mentioned this pull request Sep 26, 2017
7 tasks
@zimbatm
Copy link
Member Author

zimbatm commented Sep 26, 2017

anyone with a nvidia card wants to test this?

@fpletz fpletz changed the base branch from release-17.09 to master September 27, 2017 22:16
@fpletz fpletz merged commit 8237fa4 into NixOS:master Sep 27, 2017
@fpletz
Copy link
Member

fpletz commented Sep 27, 2017

Should be fine. Thanks!

@zimbatm zimbatm deleted the gdm-on-nvidia branch September 27, 2017 22:27
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.

None yet

7 participants