Hardcoded paths in desktop files need to be processed need $SNAP prefixing

[Charles Kerr]

Noticed and reported by mterry in several indicator patches, e.g. <https://code.launchpad.net/~mterry/indicator-sound/snap-root/+merge/308125>:

> There are also two hardcoded paths in the data/*.desktop.in files.
> But that's a static desktop file, so $SNAP isn't appropriate.
> I think it makes more sense to update the code that reads those
> files to do the prefixing.

There is also an umbrella ticket for hardcoded paths in the u8 snap: <https://pad.lv/1629009>

[dobey (dobey)]

I'm not sure how much of this is valid. Most of these are .desktop files that are installed in the XDG autostart directory, and thus only used in environments which use that, of which Unity8 is not one, especially so in its snap form.

I suppose this also begs the question of whether Unity 8 should implement the XDG autostart spec?

[Michael Terry (mterry)]

Let's not transform this bug into a feature request for u8. :)

(Though just converting autostart files to systemd user-units is probably a better use of our time?)

But your point about not worrying about converting .desktop files that are only used for xdg-autostarting makes sense.

[dobey (dobey)]

Yeah, the indicators all ship upstart jobs, systemd units, and autostart .desktop files.

There are some cases where indicators need fixed in other ways to deal with similarly related issues (parsing other .desktop files), but generally the data/*.desktop.in files aren't a problem here, from what I can see.

[Launchpad Janitor (janitor)]

This bug was fixed in the package indicator-transfer - 0.2+17.04.20161205-0ubuntu1

---------------
indicator-transfer (0.2+17.04.20161205-0ubuntu1) zesty; urgency=medium

  * Use cmake-extras EnableCoverageReport instead of custom code for
    coverage.
  * Load plug-ins from $SNAP path too.
  * Use UAL C++ API to get app info and launch apps. (LP: #1638708)

 -- Rodney Dawes <email address hidden> Mon, 05 Dec 2016 18:03:02 +0000