* lisp/shell.el (shell-directory-tracker): Go back to just ignoring failures.

Fixes: debbugs:17159

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 7fbb5cb960e3b6fa98137a15079488355737d2f0..26004f8d6e2e86c1661f9ac2a7f84f71a9685cfb 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-04-07  Glenn Morris  <rgm@gnu.org>
+
+       * shell.el (shell-directory-tracker):
+       Go back to just ignoring failures.  (Bug#17159)
+
 2014-04-06  João Távora  <joaotavora@gmail.com>
 
        Fix `electric-pair-delete-adjacent-pairs' in modes binding

diff --git a/lisp/shell.el b/lisp/shell.el
index c5c1275f19f9cd58dc7c2781739d6fa546db6a9f..9dc9a013f8b23a77dfe7c75ad434911a449bfc91 100644 (file)
--- a/lisp/shell.el
+++ b/lisp/shell.el
@@ -792,8 +792,11 @@ and `shell-pushd-dunique' control the behavior of the relevant command.
 Environment variables are expanded, see function `substitute-in-file-name'."
   (if shell-dirtrackp
       ;; We fail gracefully if we think the command will fail in the shell.
-      (with-demoted-errors "Couldn't cd: %s"
-         (let ((start (progn (string-match
+;;;      (with-demoted-errors "Directory tracker failure: %s"
+      ;; This fails so often that it seems better to just ignore errors (?).
+      ;; Eg even: foo=/tmp; cd $foo is beyond us (bug#17159).
+      (ignore-errors
+        (let ((start (progn (string-match
                               (concat "^" shell-command-separator-regexp)
                               str) ; skip whitespace
                              (match-end 0)))