(browse-url): Set DISPLAY to the one of the

current frame, in case we're connected to several displays.

2 files changed, 27 insertions, 16 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 4b02e94cc47..c641766360d 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2006-12-08  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+        * net/browse-url.el (browse-url): Set DISPLAY to the one of the
+        current frame, in case we're connected to several displays.
+
 2006-12-08  Juanma Barranquero  <lekktu@gmail.com>
 
         * frame.el (other-frame): Doc fix.
@@ -20,9 +25,10 @@
 2006-12-05  Kim F. Storm  <storm@cua.dk>
 
         * emulation/cua-base.el (cua-paste-pop-rotate-temporarily): Doc fix.
-        (cua-paste-pop): Rework last change for cua-paste-pop-rotate-temporarily,
-        so first M-y and C-y works alike, pasting the head of the kill-ring, and
-        prefix arg C-u M-y inserts the text inserted by the last M-y command.
+        (cua-paste-pop): Rework last change for
+        cua-paste-pop-rotate-temporarily, so first M-y and C-y works alike,
+        pasting the head of the kill-ring, and prefix arg C-u M-y inserts the
+        text inserted by the last M-y command.
 
 2006-12-05  Micha,Ak(Bl Cadilhac  <michael.cadilhac@lrde.org>
 
@@ -39,8 +45,7 @@
         * progmodes/fortran.el (fortran-mode-abbrev-table)
         * progmodes/octave-mod.el (octave-abbrev-table)
         * progmodes/sql.el (sql-mode-abbrev-table): Define abbrevs even
-        if abbrev-table is non-nil (saved user abbrevs may have been
-        restored).
+        if abbrev-table is non-nil (saved user abbrevs may have been restored).
 
         * progmodes/vhdl-mode.el (vhdl-mode-abbrev-table-init): Do not
         clear abbrev table, else saved abbrevs will not be restored.
diff --git a/lisp/net/browse-url.el b/lisp/net/browse-url.el
index 3f514a2aaab..33006731622 100644
--- a/lisp/net/browse-url.el
+++ b/lisp/net/browse-url.el
@@ -760,17 +760,23 @@ Prompts for a URL, defaulting to the URL at or before point.  Variable
   (interactive (browse-url-interactive-arg "URL: "))
   (unless (interactive-p)
     (setq args (or args (list browse-url-new-window-flag))))
-  (if (functionp browse-url-browser-function)
-      (apply browse-url-browser-function url args)
-    ;; The `function' can be an alist; look down it for first match
-    ;; and apply the function (which might be a lambda).
-    (catch 'done
-      (dolist (bf browse-url-browser-function)
-        (when (string-match (car bf) url)
-          (apply (cdr bf) url args)
-          (throw 'done t)))
-      (error "No browse-url-browser-function matching URL %s"
-             url))))
+  (let ((process-environment (copy-sequence process-environment)))
+    ;; When connected to various displays, be careful to use the display of
+    ;; the currently selected frame, rather than the original start display,
+    ;; which may not even exist any more.
+    (if (stringp (frame-parameter (selected-frame) 'display))
+        (setenv "DISPLAY" (frame-parameter (selected-frame) 'display)))
+    (if (functionp browse-url-browser-function)
+        (apply browse-url-browser-function url args)
+      ;; The `function' can be an alist; look down it for first match
+      ;; and apply the function (which might be a lambda).
+      (catch 'done
+        (dolist (bf browse-url-browser-function)
+          (when (string-match (car bf) url)
+            (apply (cdr bf) url args)
+            (throw 'done t)))
+        (error "No browse-url-browser-function matching URL %s"
+               url)))))
 
 ;;;###autoload
 (defun browse-url-at-point (&optional arg)