* net/tramp-sh.el (tramp-histfile-override): Use `unset' as default.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 382b6537af6839a4265f6d5c5374bc97cce795be..6e5b078f94d700183f54b84acd7607213598d335 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -8,7 +8,8 @@
 
 2015-02-02  Michael Albinus  <michael.albinus@gmx.de>
 
-       * net/tramp-sh.el (tramp-histfile-override): Add another choice 'unset.
+       * net/tramp-sh.el (tramp-histfile-override): Add another choice
+       'unset.  Use it as default.
        (tramp-open-shell, tramp-maybe-open-connection): Support it.
        (Bug#19731)
 

diff --git a/lisp/net/tramp-sh.el b/lisp/net/tramp-sh.el
index ae0dad715bf689f718845b6cc142b755221aec90..20e32e9df6be576294f0dcc01976bf48b324ad7f 100644 (file)
--- a/lisp/net/tramp-sh.el
+++ b/lisp/net/tramp-sh.el
@@ -65,13 +65,13 @@ files conditionalize this setup based on the TERM environment variable."
   :type 'string)
 
 ;;;###tramp-autoload
-(defcustom tramp-histfile-override "/dev/null"
+(defcustom tramp-histfile-override 'unset
   "When invoking a shell, override the HISTFILE with this value.
-By default, the HISTFILE is set to the \"/dev/null\" value, which
-is special on Unix systems and indicates the shell history should
-not be logged (this avoids clutter due to Tramp commands).
-
-The symbol `unset' removes any setting of HISTFILE.
+By default, it is set to the symbol `unset', which unsets any
+setting of HISTFILE.  When setting to a string, it redirects the
+shell history to that file.  Be careful when setting to
+\"/dev/null\"; this might result in undesired results when using
+\"bash\" as shell.
 
 If you set this variable to nil, however, the *override* is
 disabled, so the history will go to the default storage
@@ -79,8 +79,7 @@ location, e.g. \"$HOME/.sh_history\"."
   :group 'tramp
   :version "25.1"
   :type '(choice (const :tag "Do not override HISTFILE" nil)
-                 (const :tag "Unset HISTFILE" 'unset)
-                 (const :tag "Empty the history (/dev/null)" "/dev/null")
+                 (const :tag "Unset HISTFILE" unset)
                  (string :tag "Redirect to a file")))
 
 ;;;###tramp-autoload