corsac at debian.org
Mon Jun 10 22:18:14 CEST 2013
On ven., 2013-06-07 at 21:31 -0700, Jeroen van Aart wrote:
> Jonas Kulla wrote:
> >> Considering the plugin is part of the core xfce apps I will assume it's
> >> the appropriate way to do it. I also found out about exo_execute_preferred_
> >> **application() however I didn't feel like including another library.
> > Why not? Exo is part of core xfce, so you'll always have it present.
> You're right, I phrased it incorrectly. I had to include another header
> file and edit makefiles to get exo library to work and at the moment I
> didn't feel like doing that. Also I felt that linking against an extra
> library wasn't such a good idea as opposed to running an external
> command the way the weather plugin does.
Running external commands directly from a plugin is what I find weird
here. Especially since you need to take care yourself of preventing
command injection and stuff like that.
Since you'll need exo installed anyway (to have exo-open), just do
everyone a favor and add the dependency.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 490 bytes
Desc: This is a digitally signed message part
More information about the Xfce4-dev