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

Bug #1638708 reported by Charles Kerr on 2016-11-02
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
indicator-application (Ubuntu)
Undecided
Unassigned
indicator-bluetooth (Ubuntu)
Undecided
Unassigned
indicator-datetime (Ubuntu)
Undecided
Unassigned
indicator-location (Ubuntu)
Undecided
Unassigned
indicator-messages (Ubuntu)
Undecided
Unassigned
indicator-power (Ubuntu)
Undecided
Unassigned
indicator-session (Ubuntu)
Undecided
Unassigned
indicator-sound (Ubuntu)
High
Unassigned
indicator-transfer (Ubuntu)
Undecided
Unassigned

Bug Description

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>

Related branches

Charles Kerr (charlesk) on 2016-11-02
description: updated
summary: - Hardcoded paths in data/*.desktop.in need Snappy prefixing
+ Hardcoded paths in desktop files need to be processed need $SNAP
+ prefixing
Charles Kerr (charlesk) on 2016-11-02
description: updated
dobey (dobey) wrote :

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) wrote :

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) wrote :

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.

dobey (dobey) on 2016-12-08
no longer affects: indicator-printers (Ubuntu)
no longer affects: indicator-network (Ubuntu)
Launchpad Janitor (janitor) wrote :

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

Changed in indicator-transfer (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers