bssh should use default terminal when launched from menu

[uws]

The summary says it all. Right now bssh (when started from the Gnome menu) launches xterm on my machine, while gnome-terminal should be used. Since ssh needs a terminal anyway, it seems better to just open the default one by specifying this in the .desktop file. The fallback mechanism is left intact for people not using the menu to start bssh.

I'll attach a patch.

[uws]

Always open the default terminal when launched from the menu

[lennart]

Are you running some kind of Debian (or derived) distro? If so you probably have misconfigured your x-terminal-emulator alternatives setting.

The relevant code in bssh.c looks like this:

execlp("x-terminal-emulator", "x-terminal-emulator", "-T", n, "-e", "ssh", "-p", p, "-l", u, h, NULL); 
execlp("gnome-terminal", "gnome-terminal", "-t", n, "-x", "ssh", "-p", p, "-l", u, h, NULL);
execlp("xterm", "xterm", "-T", n, "-e", "ssh", "-p", p, "-l", u, h, NULL);

The first execlp() call that succeeds wins. Since gnome-terminal has not been started on your machine this probably means that x-terminal-emulator was not configured correctly. bssh.c tries to follow your personal settings on your machine. It cannot do clairvoyance and determine that x-terminal-emulator was misconfigured. ;-)

[uws]

I disagree ;) I am indeed running Debian on my machine. However, where x-terminal-emulator points to is a system-wide setting. While I happen to be root on the machine I'm working on right now, this is not always the case, e.g. at the university.

Perhaps some kind of "desktop detection" would be needed? E.g. check for the GNOME_DESKTOP_SESSION_ID envvar, if found get the preferred terminal emulator from gconf (using gconftool), and finally launch it... of course my patch does exactly this, but it opens a terminal before the host is chosen (which I don't think is a problem).

[lennart]

Hmm, what would you say if add support to run xdg-terminal before I try x-terminal-emulator? That reads the config from gconf and starts the right terminal.

(No jokes about project Portland please! ;-))

[lennart]

Hmm, unfortunately I had to learn that xdg-terminal is not yet part of an official portland release. Tsss.