browse-url-galeon incorrect

[Paul Smith]

This bug report will be sent to the Free Software Foundation,
not to your local site managers!
Please write in English, because the Emacs maintainers do not have
translators to read other languages for them.

Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.

In GNU Emacs 21.2.1 (i386-debian-linux-gnu, X toolkit, Xaw3d scroll bars)
 of 2002-03-22 on raven, modified by Debian
configured using `configure  i386-debian-linux-gnu --prefix=/usr 
--sharedstatedir=/var/lib --libexecdir=/usr/lib --localstatedir=/var/lib 
--infodir=/usr/share/info --mandir=/usr/share/man --with-pop=yes --with-x=yes 
--with-x-toolkit=athena --without-gif'
Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: 
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: C
  locale-coding-system: nil
  default-enable-multibyte-characters: t

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

The browse-url.el:browse-url-galeon function is wrong.  It constructs
the arguments to Galeon like this:

      (concat "galeon " url) nil
      browse-url-galeon-program
      (append
      browse-url-galeon-arguments
      (if new-window '("-w" "--noraise"))
      (list "-x" url))

This is broken, though.  The "-x" argument overrides the setting of the
new-window flag; "-x" means "Attempt to load URL in existing Galeon
window", and the last such argument on the command line always takes
precedence.  I think the "-x" should be removed, as I think it's not
appropriate to have no way of passing _exactly_ the arguments you want,
and only those.


Recent input:
<help-echo> <switch-frame> <down-mouse-2> <mouse-2> 
<switch-frame> <help-echo> <help-echo> <switch-frame> 
C-h v b r o <tab> n <tab> w <tab> ` <backspace> <return> 
<help-echo> <down-mouse-2> <mouse-2> M-f M-f M-b C-s 
C-w C-w C-w C-w C-w C-s C-s C-s C-a C-s g a l e o n 
C-s C-s C-s C-n C-a C-n C-n C-n C-n C-n C-n C-n C-n 
C-n C-n C-l C-n C-n C-n C-n C-n C-n C-n C-n C-n C-n 
C-n C-n C-l C-e C-n C-e M-x e m a c s - <M-backspace> 
r e p o <tab> o <tab> r <backspace> r <tab> <backspace> 
<backspace> r <tab> <return>

Recent messages:
Scanning buffer for index ( 45%)
Scanning buffer for index ( 52%)
Scanning buffer for index ( 58%)
Scanning buffer for index ( 64%)
Scanning buffer for index ( 70%)
Scanning buffer for index ( 76%)
Scanning buffer for index (100%)
Mark saved where search started [2 times]
Making completion list...
Loading /usr/share/emacs/21.2/lisp/mail/emacsbug.elc...done


-- 
-------------------------------------------------------------------------------
 Paul D. Smith <pausmith@nortelnetworks.com> HASMAT: HA Software Mthds & Tools
 "Please remain calm...I may be mad, but I am a professional." --Mad Scientist
-------------------------------------------------------------------------------
   These are my opinions---Nortel Networks takes no responsibility for them.