-
Notifications
You must be signed in to change notification settings - Fork 25
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
[Linux] desktop file is lacking a Categories= entry #13
Comments
Latest release is still suffering from this. Should be easy to fix. |
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 ***@***.***> 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
<#13 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABITfY-3cDo0ffcVu2z6yEzTQfwEkkUcks5senSggaJpZM4O79tq>
.
|
Thanks @cfry. The specification is at Did you test your desktop fie using The following passes validation:
|
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 ***@***.***> 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
<#13 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABITfT0R6UTo2TY5rlU2v6X5YJwuu4hcks5sfXLkgaJpZM4O79tq>
.
|
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 |
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 ***@***.***> 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
<#13 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABITfRFErIbT76Mcjq4PcJF7wpZ00eFgks5sfiptgaJpZM4O79tq>
.
|
I think so, yes. |
The desktop file is lacking a
Categories=
entry in the*.desktop
file.According to the menu spec,
Hence, please add at least one of the following in the
Categories=
key.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: AppImage/appimage.github.io#2
The text was updated successfully, but these errors were encountered: