bug#30228: desktop files embedding full path

[Efraim Flashner]

On Wed, Jan 24, 2018 at 03:58:51PM +0100, Ludovic Courtès wrote:
> Hi,
> 
> Efraim Flashner <address@hidden> skribis:
> 
> > the owncloud-client package includes an option to add a .desktop entry
> > to $HOME/.configure/autostart/, which allows it to autostart when you
> > log in to your desktop environment. Because the full path of the binary
> > is in the .desktop file, it will always start THAT copy of
> > owncloud-client, even if it is garbage collected.
> 
> I think you shouldn’t copy manually the .desktop file to
> ~/.configure/autostart, precisely because of that.
> 
> Instead this should be done via in indirect GC root (as in “guix build
> -r the-root owncloud-client”) or, ideally, by having it GuixSD-managed.
> 
> Thoughts?
> 
> Ludo’.

I agree it shouldn't copy a .desktop file into .config/autostart/. In
this case, the offending code is here¹, so there's not much that can be
done about it. In mine I've changed the Exec line to
/var/guix/profiles/per-user/... so it'll work as expected. In this case
the code generates a desktop file on the fly, so modifying the .desktop
file shipping with applications wouldn't even help here.


¹ 
https://github.com/owncloud/client/blob/f326bf371d690f1929f7c5c06aebf53c7255daa6/src/common/utility_unix.cpp#L73

-- 
Efraim Flashner   <address@hidden>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

signature.asc
Description: PGP signature