For me, my plan for this part is to make lxshortcut a library, or have this functionality built into libfm directly.
A library might be more appropriate since this can also be used by lxsession-edit later.
A separate binary is better when we want to make it an optional component and handle cases when it's not installed.
Making it a library will let other programs integrate it nicely and easily since many other desktop components might need this functionality, such as lxpanel, pcmanfm, and lxsession-edit.
Putting it in libfm has the benefit that we don't have to maintain yet another library. The cons is other programs requiring this feature will depend on libfm.
I have not decide how to do this yet, but personally I prefer the library one. (maybe libdesktop-edit?)
Le Friday 12 August 2011 à 16:00 +0200, Leszek Lesner a écrit :
Thanks Leszek for the patch. You should post it to pcmanfm bugtracker so> I created a little patch which creates a new menuitem in the new menu called
> When its called it will execute
> lxshortcut -i <filename_you_entered_for_the_new_desktop.file>
> I originally created this patch for ZevenOS-Neptune and the debian version of
> pcmanfm (which seems to be an older version).
> I hope this patch works also for the ubuntu package and the newer version and
> is useful until a 'real' integration for desktop creation is done in pcmanfm /
upstream can have a look at it :)
I'm not sure the use case you try to cover with this patch. Do you have
an example ? Also, IMO it should handle the case when lxshortcut is not
installed (adding a text box asking to install lxshortcut).