bash: Make sure HISTFILE's directory exists

nix-community · Jan 12, 2025 · 50f55ee · 50f55ee
1 parent 2532b50
commit 50f55ee
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 2 deletions.
diff --git a/modules/programs/bash.nix b/modules/programs/bash.nix
@@ -180,7 +180,7 @@ in {
 
     sessionVarsStr = config.lib.shell.exportAll cfg.sessionVariables;
 
-    historyControlStr = concatStringsSep "\n"
+    historyControlStr = (concatStringsSep "\n"
       (mapAttrsToList (n: v: "${n}=${v}") ({
         HISTFILESIZE = toString cfg.historyFileSize;
         HISTSIZE = toString cfg.historySize;
@@ -190,7 +190,10 @@ in {
         HISTCONTROL = concatStringsSep ":" cfg.historyControl;
       } // optionalAttrs (cfg.historyIgnore != [ ]) {
         HISTIGNORE = escapeShellArg (concatStringsSep ":" cfg.historyIgnore);
-      }));
+      }))) + (optionalString (cfg.historyFile != null) ''
+
+        mkdir -p "$(dirname "$HISTFILE")"
+      '');
   in mkIf cfg.enable {
     home.packages = [ cfg.package ];
 

diff --git a/tests/modules/programs/bash/bash-history-control-with-file.nix b/tests/modules/programs/bash/bash-history-control-with-file.nix
@@ -0,0 +1,21 @@
+{ config, lib, pkgs, ... }:
+
+with lib;
+
+{
+  config = {
+    programs.bash = {
+      enable = true;
+      historyControl = [ "erasedups" ];
+      historyFile = "/home/hm-user/foo/bash/history";
+    };
+
+    nmt.script = ''
+      assertFileExists home-files/.bashrc
+
+    assertFileRegex \
+      home-files/.bashrc \
+      '^mkdir -p "\$(dirname "\$HISTFILE")"'
+    '';
+  };
+}
diff --git a/tests/modules/programs/bash/default.nix b/tests/modules/programs/bash/default.nix
@@ -2,4 +2,5 @@
   bash-completion = ./completion.nix;
   bash-logout = ./logout.nix;
   bash-session-variables = ./session-variables.nix;
+  bash-history-control-with-file = ./bash-history-control-with-file.nix;
 }