nixos: disable sound by default, if stateVersion > 18.03

NixOS · Feb 22, 2018 · ea48c41 · ea48c41
1 parent d328ba7
commit ea48c41
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/nixos/doc/manual/release-notes/rl-1803.xml b/nixos/doc/manual/release-notes/rl-1803.xml
@@ -209,6 +209,9 @@ following incompatible changes:</para>
       For <literal>stateVersion = "17.09"</literal> or lower the old behavior is preserved.
     </para>
     <itemizedlist>
+      <listitem>
+         <literal>sound.enable</literal> now defaults to false.
+      </listitem>
       <listitem>
         <para>
           <literal>matrix-synapse</literal> uses postgresql by default instead of sqlite.

diff --git a/nixos/modules/installer/tools/nixos-generate-config.pl b/nixos/modules/installer/tools/nixos-generate-config.pl
@@ -603,6 +603,10 @@ sub multiLineList {
   # Enable CUPS to print documents.
   # services.printing.enable = true;
 
+  # Enable sound.
+  # sound.enable = true;
+  # hardware.pulseaudio.enable = true;
+
   # Enable the X11 windowing system.
   # services.xserver.enable = true;
   # services.xserver.layout = "us";

diff --git a/nixos/modules/services/audio/alsa.nix b/nixos/modules/services/audio/alsa.nix
@@ -21,7 +21,6 @@ in
 
       enable = mkOption {
         type = types.bool;
-        default = true;
         description = ''
           Whether to enable ALSA sound.
         '';
@@ -78,7 +77,9 @@ in
 
   ###### implementation
 
-  config = mkIf config.sound.enable {
+  config = {
+      sound.enable = mkDefault !(versionAtLeast config.system.stateVersion "18.03")
+    } // mkIf config.sound.enable {
 
     environment.systemPackages = [ alsaUtils ];