cfry / dde

Dexter Development Environment
Other
44 stars 25 forks source link

[Linux] desktop file is lacking a Categories= entry #13

Open probonopd opened 7 years ago

probonopd commented 7 years ago

The desktop file is lacking a Categories= entry in the *.desktop file.

According to the menu spec,

By including one of the Main Categories in an application's desktop entry file, the application will be ensured that it will show up in a section of the application menu dedicated to this category. If multiple Main Categories are included in a single desktop entry file, the entry may appear more than once in the menu.

Hence, please add at least one of the following in the Categories= key.

Main Category Description Notes
AudioVideo Application for presenting, creating, or processing multimedia (audio/video)
Audio An audio application Desktop entry must include AudioVideo as well
Video A video application Desktop entry must include AudioVideo as well
Development An application for development
Education Educational software
Game A game
Graphics Application for viewing, creating, or processing graphics
Network Network application such as a web browser
Office An office type application
Science Scientific software
Settings Settings applications Entries may appear in a separate menu or as part of a "Control Center"
System System application, "System Tools" such as say a log viewer or network monitor
Utility Small utility application, "Accessories"

In addition, you could specify one or more from the longer list of Additional Categories.

Please test the result with desktop-file-validate and make sure it passes.

electron-builder offers native support for this since v19.22.1.

Reference: https://github.com/AppImage/AppImageHub/issues/2

probonopd commented 7 years ago

Latest release is still suffering from this. Should be easy to fix.

cfry commented 7 years ago

Your email is not sufficient to tell me how to do this. I have constructed a file named .desktop put it in the top level folder of my code, with the content:

for linux, required by github

Type=Application Name=Dexter Development Environment Exec=dexter_dev_env Categories=Development,Education,Robotics

Is that sufficient?

After searching for 20 minutes, I found

no documentation on the web as to the

separator character between categories,

so I'm guessing its comma.

This will be in my next release.

On Sun, Sep 3, 2017 at 5:42 AM, probonopd notifications@github.com wrote:

Latest release is still suffering from this. Should be easy to fix.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/cfry/dde/issues/13#issuecomment-326794662, or mute the thread https://github.com/notifications/unsubscribe-auth/ABITfY-3cDo0ffcVu2z6yEzTQfwEkkUcks5senSggaJpZM4O79tq .

probonopd commented 7 years ago

Thanks @cfry. The specification is at https://standards.freedesktop.org/menu-spec/latest/apa.html

Did you test your desktop fie using desktop-file-validate?

The following passes validation:

[Desktop Entry]
Type=Application
Name=Dexter Development Environment
Comment=Develop and run software on the Dexter 5 axis robot arm
Exec=dexter_dev_env
Categories=Education;Robotics
cfry commented 7 years ago

Did you test your desktop fie using desktop-file-validate? No, I don't know how. I don't even have a linux machine, I do test on Windows and Mac which don't have the notion of a desktop file. I see that Categories needs a semicolon between items. Thanks!!!

PS: if this is a github requirement, how about if github hosts the desktop-file-validate tool? Maybe it even runs the tool automatically and emails the github author with any errors plus a link that has documentation (like what's the separator between categories -: ).

On Tue, Sep 5, 2017 at 12:12 PM, probonopd notifications@github.com wrote:

The specification is at https://standards.freedesktop.org/menu-spec/latest/apa.html

Did you test your desktop fie using desktop-file-validate?

The following passes validation:

[Desktop Entry] Type=Application Name=Dexter Development Environment Comment=Develop and run software on the Dexter 5 axis robot arm Exec=dexter_dev_env Categories=Education;Robotics

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cfry/dde/issues/13#issuecomment-327225821, or mute the thread https://github.com/notifications/unsubscribe-auth/ABITfT0R6UTo2TY5rlU2v6X5YJwuu4hcks5sfXLkgaJpZM4O79tq .

probonopd commented 7 years ago

Thanks cfry. This is not a GitHub requirement but a Desktop Linux requirement. electron-builder allows you to add a category, and it will write the desktop file for you. Please see the category key at https://www.electron.build/configuration/linux-other

cfry commented 7 years ago

OK, good. So this means I can get rid of my .desktop file after I've added

"build": { "appId": "com.haddington.dde", "linux": { "category": "Development;Education;Robotics" } }

to my package.json file?

Thanks for your help.

On Wed, Sep 6, 2017 at 1:15 AM, probonopd notifications@github.com wrote:

Thanks cfry. This is not a GitHub requirement but a Desktop Linux requirement. electron-builder allows you to add a category, and it will write the desktop file for you. Please see the category key at https://www.electron.build/configuration/linux-other

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cfry/dde/issues/13#issuecomment-327376949, or mute the thread https://github.com/notifications/unsubscribe-auth/ABITfRFErIbT76Mcjq4PcJF7wpZ00eFgks5sfiptgaJpZM4O79tq .

probonopd commented 7 years ago

I think so, yes.