Support for tryExec in sidebar plugins

Marcus Camen mcamen at mcamen.de
Mon Jul 19 20:05:50 BST 2004


On Monday 19 July 2004 20:43, David Faure wrote:
> On Monday 19 July 2004 20:33, Marcus Camen wrote:
> > Hi *,
> >
> > AFAIK for kdm session files (like twm.desktop) we read the 'TryExec'
> > key in the .desktop file and check if the given executable is
> > available or not. The same applies for KService which is used by the
> > Kontact plugin system (e.g. David and Daniel committed a modified
> > newsticker plugin today).
> >
> > The attached patch implements a similar functionality in
> > kdebase/konqueror/sidebar/sidebar_widget.cpp. It affects the 'Add
> > New' popup menu in a way that sidebar plugins which have
> > (TryExec==foo && executable foo does not exist) will not be shown.
> > Therefore a user will not be able to add non-functional plugins.
>
> I like the idea but I think it should be implemented with
> KDesktopFile::tryExec()

Yeah, I ripped from kdm/kfrontend/kgreeter.h  ;)
Patch with KDesktopFile is attached.

-- 
Marcus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sidebar-KDesktopFile.diff
Type: text/x-diff
Size: 1511 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040719/b4a14475/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040719/b4a14475/attachment.sig>


More information about the kde-core-devel mailing list