-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
nixos/console: Fix systemd-vconsole-setup #299456
base: master
Are you sure you want to change the base?
Conversation
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.
The current implementation does not solve the issue. The same error still occurs.
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.
Works for me, diff LGTM.
Tested by adding the following to my config:
{pkgs, config, ...}:
{
console.font = "Lat2-Terminus16";
boot.initrd.systemd.contents."/etc/kbd/consolefonts" =
let
cfg = config.console;
consoleEnv = kbd: pkgs.buildEnv {
name = "console-env";
paths = [ kbd ] ++ cfg.packages;
pathsToLink = [
"/share/consolefonts"
"/share/consoletrans"
"/share/keymaps"
"/share/unimaps"
];
};
in
lib.mkIf (!cfg.earlySetup && cfg.font != null) { source = "${consoleEnv config.boot.initrd.systemd.package.kbd}/share/consolefonts"; };
}
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.
Working fine! 馃槂 馃憤
Maybe consider squashing and prefix the commit message with "nixos/console:"
About the commit number and message/name/title that lelgenio commented: https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions |
b1f231d
to
6140568
Compare
Sorry, had missed that message. Should be fixed now! |
Isn't this essentially undoing 18363ca? It seems to me that now you're always setting the console with systemd initrd. |
I still really want to fix this, as it's causing me trouble but don't know how to proceed. Can someone more familiar with these modules give some pointers on how to resolve this? |
@dasJ ? |
Description of changes
Fixes the problem described in #257904
Fixes #257904
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.