desktop file Exec mangling broken

[Harald Sitter]

According to https://docs.snapcraft.io/the-snap-format/698 the Exec line of desktop files should be in the form $snap.$app. Snapcraft apparently doesn't mangle desktop files it picks up via `apps.$app.desktop` or appstream magic though.

Example https://github.com/apachelogger/snap-appstreamtest

```
➜ grep Exec prime/meta/gui/kbruch.desktop
Exec=kbruch %U
```

[Claudio Matsuoka (cmatsuoka)]

In the provided example, the suffix is omitted because the application name (kbruch) matches the snap name. If we rename the snap to something else and build the package, the line in the desktop file becomes:

$ snapcraft --destructive-mode
Pulling kbruch
Building kbruch
Staging kbruch
Priming kbruch
Snapping 'somethingelse' |
Snapped somethingelse_18.12.1_amd64.snap
$ grep Exec prime/meta/gui/kbruch.desktop
Exec=somethingelse.kbruch %U

which I believe is the expected behavior for rewriting desktop entries. Am I missing something?

[Harald Sitter (apachelogger)]

Perhaps I am imagining things, but I could have sworn the documentation didn't say anything about the app portion getting omitted when the names are the same when I filed this bug.

Anyway, seeing as the docs line up with how snapcraft behaves this is a non-issue.