search

jlambert360 / FPM-Installer

THIS SCRIPT IS OUTDATED PLEASE USE THE APPIMAGE REPO INSTEAD.
https://github.com/jlambert360/FPM-AppImage/releases/latest
8 stars 6 forks source link

changed '-' in desktop file name to '_', changed Version key to speci… #13

Closed rishvic closed 3 years ago

rishvic commented 4 years ago

…fy Desktop Spec version, added TryExec key

While reading the desktop specification, I saw some small discrepancies between the desktop file being added, and the desktop specification. So, just fixed some things in the setup scripts

Changes

  1. Changed "Version" key to specify Desktop Specification Version, as is needed
  2. Added a "TryExec" key with the value being the path of the executable
  3. Changed file name of the desktop file to contain underscores instead of dashes, as specified in the Desktop Spec

Quote from the Desktop Specification:

... Well-known names containing the dash are allowed but not recommended, because the dash is not allowed in some related uses of reversed DNS names, such as D-Bus object paths and interface names, and Flatpak app IDs. If the author's domain name contains a dash, replacing it with an underscore is recommended: this cannot cause ambiguity, because underscores are not allowed in DNS domain names. ...

Link to the desktop specification I used: https://specifications.freedesktop.org/desktop-entry-spec/latest/ (As of 26 July 2020)