bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: browse-url-galeon incorrect


From: Simon Josefsson
Subject: Re: browse-url-galeon incorrect
Date: Wed, 15 May 2002 20:04:20 +0200
User-agent: Gnus/5.090007 (Oort Gnus v0.07) Emacs/21.2.50 (i686-pc-linux-gnu)

"Paul Smith"<address@hidden> 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)))))
...




reply via email to

[Prev in Thread] Current Thread [Next in Thread]