-
Notifications
You must be signed in to change notification settings - Fork 17.4k
🐧 .desktop file without capital letter #3803
Conversation
The .desktop file of Atom was named Atom.desktop but it should be renamed to atom.desktop because a .desktop should not contain capital letters as any other .desktop files in /usr/share/applications. Note that without that, it can be not easy to link the window with the .desktop file and then a dock/panel could not link a launcher with its window.
tmpDir = if process.env.TMPDIR? then process.env.TMPDIR else '/tmp' | ||
if installDir.indexOf(tmpDir) isnt 0 | ||
desktopFile = path.join('resources', 'linux', 'Atom.desktop.in') | ||
desktopInstallFile = path.join(installDir, 'share', 'applications', 'Atom.desktop') | ||
desktopFile = path.join('resources', 'linux', 'atom.desktop.in') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This input file is checked in here: https://github.com/atom/atom/blob/master/resources/linux/Atom.desktop.in
So it needs to be renamed as well in the repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello,
I'm not sure to understand what's wrong: this file has also been renamed:
resources/linux/Atom.desktop.in → resources/linux/atom.desktop.in
Or should I miss something else? :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry, I missed that in the diff.
Thanks for this 🚢 |
🐧 .desktop file without capital letter
No problem :-) |
Hello,
Thank you for developing and maintaining Atom!
I'm using it on GNU/Linux Ubuntu and it seems the launcher (
.desktop
file) has a capital letter in its name. It should be renamed toatom.desktop
because a.desktop
file should not contain capital letters as any other.desktop
files that we can find in/usr/share/applications
.Note that we also have another problem because currently, the window's class of all Atom windows is
Atom Shell
but the launcher is namedAtom
. We should also add this line in your.desktop
file:Because without that, it's not possible for a dock/panel (like Cairo-Dock) to link a launcher (
Atom
) with its window (Atom Shell
).But it's certainly better to rename the window's class and this is why I just created this pull request in the
atom-shell
project! => electron/electron#698Best Regards,
Matt