From 8cfcb91824b60e2574203b3e4cc9c8f01b1b9ca6 Mon Sep 17 00:00:00 2001
From: Malo Bourgon <mbourgon@gmail.com>
Date: Mon, 14 Dec 2020 19:28:31 -0800
Subject: [PATCH] Change noAutoUpdate option to autoUpdate in brew-bundle
 module

---
 modules/programs/brew-bundle.nix | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/modules/programs/brew-bundle.nix b/modules/programs/brew-bundle.nix
index 295a07657..f18c299af 100644
--- a/modules/programs/brew-bundle.nix
+++ b/modules/programs/brew-bundle.nix
@@ -29,9 +29,8 @@ let
   );
 
   brew-bundle-command =
-    "HOMEBREW_NO_AUTO_UPDATE=" +
-    (if cfg.noAutoUpdate then "1" else "0") +
-    " brew bundle --file='${brewfile}' --no-lock" +
+    (if cfg.autoUpdate then "" else "HOMEBREW_NO_AUTO_UPDATE=1 ") +
+    "brew bundle --file='${brewfile}' --no-lock" +
     (if cfg.cleanup == "uninstall" || cfg.cleanup == "zap" then " --cleanup" else "") +
     (if cfg.cleanup == "zap" then " --zap" else "");
 
@@ -47,13 +46,12 @@ in
       installation instructions: https://brew.sh
     '';
 
-    noAutoUpdate = mkOption {
+    autoUpdate = mkOption {
       type = types.bool;
-      default = true;
-      example = false;
+      default = false;
       description = ''
-        Sets the <literal>HOMEBREW_NO_AUTO_UPDATE</literal> environment variable when running the
-        <command>brew bundle</command> command. The default is <literal>true</literal> so that
+        When enabled, sets the <literal>HOMEBREW_NO_AUTO_UPDATE</literal> environment variable
+        during `nix-darwin` activation. The default is <literal>false</literal> so that
         repeated invocations of <command>darwin-rebuild switch</command> are idempotent.
       '';
     };