| 25 Nov 2022 |
 K900 | users.users.k900.shell = let
wrapper = pkgs.writeShellScriptBin "shell-wrapper" ''
. /etc/set-environment
exec ${pkgs.zsh}/bin/zsh "$@"
'';
in lib.mkForce "${wrapper}/bin/shell-wrapper";
| 18:08:47 |
| K900 | This works. | 18:08:49 |
| K900 | I hate it. | 18:08:51 |
| K900 | https://github.com/nix-community/NixOS-WSL/pull/170 | 18:10:26 |
| K900 | This turned out to be unrelated but still is more correct | 18:10:34 |
| K900 | Also nzbr (they/it) while you're here pls merge https://github.com/nix-community/NixOS-WSL/pull/169 | 18:10:53 |
 Arson Copperflame | Can you have a look at #161? I want to have the tests merged before the other PRs - they can't do native systemd yet, but we can at least prevent breaking things that already work | 18:13:14 |
| K900 | https://github.com/microsoft/WSL/issues/9213 | 18:14:57 |
| K900 | Opened | 18:14:58 |
| K900 | In reply to @nzbr:nzbr.de
Can you have a look at #161? I want to have the tests merged before the other PRs - they can't do native systemd yet, but we can at least prevent breaking things that already work My PowerShell is terrible but seems OK to me | 18:15:19 |
| Arson Copperflame | I mean, the tests work xD | 18:16:48 |
| K900 | Then just merge them? | 18:18:05 |
| K900 | If it breaks, we can always fix it | 18:18:15 |
| Arson Copperflame | Merged! Please rebase/merge main into your PRs K900 Sandro 🐧 psvo | 18:22:06 |
| K900 | Done | 18:22:35 |
| K900 | Also filed an issue on our side: https://github.com/nix-community/NixOS-WSL/issues/171 | 18:22:41 |
| K900 | Oh no I found another bug | 18:23:57 |
| K900 | The installer does not work AT ALL with native-systemd | 18:24:24 |
| K900 | Because we're propagating /etc/wsl.conf into it | 18:25:10 |
| K900 | And there's obviously no systemd in the installer | 18:25:17 |
| Arson Copperflame | I know, that's why I skipped making tests for it for now | 18:25:17 |
| Arson Copperflame | We need to include /etc/wsl.conf for the initial boot to work correctly | 18:25:36 |
| K900 | Maybe it should be a different wsl.conf then | 18:25:55 |
| Arson Copperflame | I'd suggest we run the installer script through the systemd wrapper and include that in the installer tarball | 18:25:57 |
| K900 | That would need us to stick the entire systemd closure into the initial tarball | 18:26:22 |
| K900 | Which is oof | 18:26:25 |
| K900 | What's the smallest wsl.conf we can have that still works? | 18:26:47 |
| K900 | Maybe we should just have the installer read -p at the end and die horribly | 18:27:10 |
| K900 | And then make you restart the VM | 18:27:16 |
| Arson Copperflame | We could make a second binary that execs the systemd wrapper once it's unpacked | 18:28:34 |
