Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Doesn't report .desktop file to clients #5

Open
robert-ancell opened this issue Apr 5, 2018 · 6 comments
Open

Doesn't report .desktop file to clients #5

robert-ancell opened this issue Apr 5, 2018 · 6 comments

Comments

@robert-ancell
Copy link

Due to meta/snap/telegramdesktop.desktop not matching the snap name it does't show to snapd clients (e.g. GNOME Software).

@robert-ancell
Copy link
Author

The simplest solution would be to rename the .desktop file - not sure if this would cause any other issues.

@3v1n0
Copy link
Owner

3v1n0 commented Apr 5, 2018

This is actually a tricky situation that is bothering me since long time... As the desktop id of the app is telegramdesktop and if I would change that it would break the launcher integration, while I didn't want to rename the desktop itself. The package also should be telegram-desktop to match to what is done everywhere, and so (while my initial name was telegramdesktop for that) I didn't change it

One thing I'd like to do is adding an alias to telegramdesktop so that I could rename the binary too, while keeping the snap as it is. But not sure if GNOME software can then reuse this.

Otherwise, one thing we could do is adding an X- parameter to the .desktop file saying what's the snap name it belongs to. As this might not work all the times.

@robert-ancell
Copy link
Author

robert-ancell commented Apr 5, 2018

Perhaps this can be solved in snap/snapcraft. From the clients point of view the name of the .desktop file does't matter, it's just snapd not connecting the "app" to the file. Could you comment on the forum about your issue and perhaps suggest the YAML could contain an explicit link?

@robert-ancell
Copy link
Author

Hah! Seems you've already done it :)

@jbicha
Copy link

jbicha commented Apr 6, 2018

@3v1n0
Copy link
Owner

3v1n0 commented Apr 6, 2018

@jbicha telegram desktop already has a provides stanza: https://github.com/telegramdesktop/tdesktop/blob/dev/lib/xdg/telegramdesktop.appdata.xml#L22

And for some random reason I was just adding this to the snap right now. However I'm not sure if the snap can already parse these informations.

@3v1n0 3v1n0 closed this as completed in 0024ecb May 7, 2018
@3v1n0 3v1n0 reopened this May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants