* browse-url-galeon incorrect
@ 2002-05-14 14:14 Paul Smith
2002-05-15 18:04 ` Simon Josefsson
0 siblings, 1 reply; 7+ messages in thread
From: Paul Smith @ 2002-05-14 14:14 UTC (permalink / raw)
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.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: browse-url-galeon incorrect
2002-05-14 14:14 browse-url-galeon incorrect Paul Smith
@ 2002-05-15 18:04 ` Simon Josefsson
2002-05-15 18:35 ` Sunil Patel
0 siblings, 1 reply; 7+ messages in thread
From: Simon Josefsson @ 2002-05-15 18:04 UTC (permalink / raw)
Cc: bug-gnu-emacs
"Paul Smith"<pausmith@nortelnetworks.com> writes:
> 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.
The current code has been changed into the following. Does it work
better for you?
...
(process (apply 'start-process
(concat "galeon " url) nil
browse-url-galeon-program
(append
browse-url-galeon-arguments
(if (browse-url-maybe-new-window new-window)
(if browse-url-galeon-new-window-is-tab
'("--new-tab")
'("--new-window" "--noraise"))
'("--existing"))
(list url)))))
...
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2002-05-17 19:50 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-05-14 14:14 browse-url-galeon incorrect Paul Smith
2002-05-15 18:04 ` Simon Josefsson
2002-05-15 18:35 ` Sunil Patel
2002-05-16 16:20 ` Simon Josefsson
2002-05-17 19:44 ` Christopher S. Kush
2002-05-17 19:50 ` Sunil Patel
2002-05-16 20:24 ` Richard Stallman
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).