On Mon, 2003-01-06 at 05:40, Stephen J. Turnbull wrote:
>>>>> "Ville" == Ville Skytt <Ville>
writes:
Ville> This should be changed in browse-url-xemacs.el and
Ville> lisp/menubar-items.el of 21.4.
"This" needs to be completely rethought IMHO.
Maybe someone should write the "locate-executable" utility that David
"I Have No Time For This" Kastrup needs. While they're at it, they
should write a "grok-executable" function to find out what options it
supports. Then we can replace half of autoconf with XEmacs!
What's the scope of "this"? Are you referring to only the
executable-find part? The duplicate locations for definitions has
already been addressed in 21.5.
Maybe it's safest just to change the mozilla function mapping in 21.4
for now. A better way would be to ditch that stuff from core
altogether, and let browse-url-xemacs handle it.
But how is browse-url-xemacs supposed to work? I'd imagine that it
would create the "Options->Internet->Open URLs with..." menu in an
autoload. However, it doesn't, the menu appears only when
browse-url-xemacs is loaded. This is for 21.1 and 21.5. On 21.4
loading it replaces the stuff in the "core" menu.
Simon, it also looks like the executable-find stuff you added for the
mozilla entry will cause errors because browse-url-mozilla-program isn't
necessarily defined at that point (ie. if browse-url hasn't been
loaded). Maybe all the *-program defcustoms in browse-url should have
an autoload cookie, only galeon and generic have it now.
Anyway, it would be nice if browse-url(-xemacs) had a list of known
browsers and their functions; this could be used in psgml-html's
"HTML->View in XYZ" menu entries instead of the current hardcoded
Netscape, W3 and kfm choices. I'm sure HTML writers would appreciate
that.
--
\/ille Skyttä
scop at
xemacs.org