[Goodies-dev] [Bug 6446] Change the associated command from 'xterm top' to 'xfce4-taskmanager'

[bugzilla-daemon at xfce.org]

http://bugzilla.xfce.org/show_bug.cgi?id=6446

--- Comment #5 from Mike Massonnet <mmassonnet at xfce.org> 2010-05-29 16:43:42 UTC ---
Created an attachment (id=2999)
 --> (http://bugzilla.xfce.org/attachment.cgi?id=2999)
cpugraph-fallback-command.patch

(In reply to comment #4)
> Now, for your proposal of setting it as a default, and changing the setting to
> something else if it fails to run, that would technically work, but that would
> also be very unintuitive. Settings changing themselves isn't expected by users.

Not the best what one can come out with :) Lets call this a little crap.

> Instead, how about changing the default command to this:
> 
> sh -c "if which xfce4-taskmanager; then xfce4-taskmanager; else exo-open
> --launch TerminalEmulator top; fi"

Nah, this is just gross to have within a text entry.

I attach a patch that does it simple-clean, if the configuration file doesn't
exist yet, the AssociatedCommand isn't set, so lets pick up this moment to
check for the right fallback command. If the program xfce4-taskmanager is not
found, let's go with top :)  Take care to test the patch, cause I didn't.

-- 
Configure bugmail: http://bugzilla.xfce.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.