Merge pull request #300 from nix-community/fixups

fix: fix config generation for legacy/tests

Author: K900

modules/build-tarball.nix

@@ -21,7 +21,7 @@ let
 
       wsl.enable = true;
       wsl.defaultUser = "nixos";
-      ${cfg.extraTarballConfig}
+      ${lib.optionalString (!cfg.nativeSystemd) "wsl.nativeSystemd = false;"}
 
       # This value determines the NixOS release from which the default
       # settings for stateful data, like file locations and database versions
@@ -34,12 +34,6 @@ let
   '';
 in
 {
-  options.wsl.extraTarballConfig = mkOption {
-    type = types.str;
-    internal = true;
-    default = "";
-  };
-
   # These options make no sense without the wsl-distro module anyway
   config = mkIf cfg.enable {
     system.build.tarballBuilder = pkgs.writeShellApplication {

tests/docker/docker-native.nix

@@ -4,6 +4,8 @@
   ];
 
   wsl.enable = true;
+  wsl.nativeSystemd = false;
+
   users.users.nixos.extraGroups = [ "docker" ];
 
   virtualisation.docker = {

tests/login-shell/session-variables.nix

@@ -9,6 +9,7 @@ in
   ];
 
   wsl.enable = true;
+  wsl.nativeSystemd = false;
 
   home-manager.users.nixos = { ... }: {
     home = {

tests/shells.Tests.ps1

@@ -18,6 +18,7 @@ Describe "Shells" {
           config = mkMerge [
             {
               wsl.enable = true;
+              wsl.nativeSystemd = false;
               users.users.nixos.shell = pkgs.$package;
             }
             (optionalAttrs (hasAttrByPath ["programs" "$package" "enable"] options) {

tests/username-change/username-change.nix

@@ -4,5 +4,6 @@
   ];
 
   wsl.enable = true;
+  wsl.nativeSystemd = false;
   wsl.defaultUser = "different-name";
 }