Veto.
I am absolutely, positively sure that there's a version of browse-url out there that
fixes
this. In fact, I've already seen this fix been put into browse-url *numerous* times.
Someone must have reverted our version, so let's fix that instead.
ben
Matt Curtin wrote:
Sending URLs to Netscape barfs if there are commas in the URL (as
there often are at
news.com,
britannica.com, and elsewhere). I added
a function to hexify the commas so it will work properly. Patch is
attached.
(brwose-url.el seems to be abandoned; the URL in the comments is dead
and nothing on the site seems to point to the new place. If it really
is abandoned, I'll act as the maintainer.)
--
Matt Curtin cmcurtin(a)interhack.net
http://www.interhack.net/people/cmcurtin/
Paula Corbin Jones David John Oates DES munitions Panama Saddam
Hussein Treasury Monica Lewinsky FBI class struggle SEAL Team 6 NWO
Vickie Weaver Kibo Noriega
------------------------------------------------------------------------
*** browse-url.el Thu Mar 4 23:53:37 1999
--- /home/cmcurtin/src/emacs/browse-url.el Tue Nov 9 13:20:25 1999
***************
*** 5,11 ****
;; Author: Denis Howe <dbh(a)doc.ic.ac.uk>
;; Maintainer: Denis Howe <dbh(a)doc.ic.ac.uk>
;; Created: 03 Apr 1995
! ;; Version: 0.38 18 Jun 1996
;; Keywords: hypertext
;; X-Home page:
http://wombat.doc.ic.ac.uk/
--- 5,11 ----
;; Author: Denis Howe <dbh(a)doc.ic.ac.uk>
;; Maintainer: Denis Howe <dbh(a)doc.ic.ac.uk>
;; Created: 03 Apr 1995
! ;; Version: 0.38.1 9 Nov 1999 (cmcurtin patch to 0.38)
;; Keywords: hypertext
;; X-Home page:
http://wombat.doc.ic.ac.uk/
***************
*** 342,347 ****
--- 342,352 ----
;; b-u-file-url encodes fewer chars. Multi-display support for
;; Netscape. Thanks Richard Mlynarik <mly(a)adoc.xerox.com>
+ ;; 0.38.1 9 Nov 1999
+ ;; browse-url-escape-url-for-netscape added to hexify commas
+ ;; in URLs. -- Matt Curtin <cmcurtin(a)interhack.net>
+
+
;;; Code:
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
***************
*** 676,681 ****
--- 681,688 ----
process-environment)
process-environment)))
+ (defun browse-url-escape-url-for-netscape (url)
+ (replace-in-string url "," "%2c" t))
;;;###autoload
(defun browse-url-netscape (url &optional new-window)
***************
*** 699,705 ****
(append browse-url-netscape-arguments
(if new-window '("-noraise"))
(list "-remote"
! (concat "openURL(" url
(if new-window ",new-window")
")"))))))
(set-process-sentinel process
--- 706,713 ----
(append browse-url-netscape-arguments
(if new-window '("-noraise"))
(list "-remote"
! (concat "openURL("
! (browse-url-escape-url-for-netscape url)
(if new-window ",new-window")
")"))))))
(set-process-sentinel process