-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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.tests.keymap: use new name of the colemak/en-latin9 keymap #50823
Conversation
Commit f1987fb renamed colemak/en-latin9 to colemak/colemak, but the keymap test wasn't adjusted to refer to the new path.
(FYI @plutotulp @Mic92 author/commiter of the commit causing the regression -- I'm guessing you might have context to review this.) |
@GrahamcOfBorg test keymap.colemak |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.keymap.colemak Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: tests.keymap.colemak Partial log (click to expand)
|
Verified locally with restrict-eval leaving access to the imported nixpkgs. Thanks! |
@GrahamcOfBorg build nixosTests.keymap.colemak this is the current workaround for ofborg. |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: nixosTests.keymap.colemak Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: nixosTests.keymap.colemak Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: nixosTests.keymap.colemak Partial log (click to expand)
|
Commit f1987fb renamed colemak/en-latin9 to
colemak/colemak, but the keymap test wasn't adjusted to refer to the new path.
Motivation for this change
Tests regressed on 2018-10-19: https://hydra.nixos.org/build/83028812
Locally reproduced with
nix-build '<nixpkgs/nixos/release-combined.nix>' -A nixos.tests.keymap.colemak.x86_64-linux
-- fails on master, passes with this PR.