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

Invalid .desktop file #500

Closed
pypingou opened this issue Apr 19, 2015 · 5 comments
Closed

Invalid .desktop file #500

pypingou opened this issue Apr 19, 2015 · 5 comments

Comments

@pypingou
Copy link
Member

While building 0.6.0 for Fedora I ran into:

desktop-file-install --vendor= --delete-original --dir=~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications ~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop
~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop: error: file contains key "_Name" in group "Desktop Entry", but key names must contain only the characters A-Za-z0-9- (they may have a "[LOCALE]" postfix)
~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop: error: file contains key "_Comment" in group "Desktop Entry", but key names must contain only the characters A-Za-z0-9- (they may have a "[LOCALE]" postfix)
~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop: error: file contains key "_Keywords" in group "Desktop Entry", but key names must contain only the characters A-Za-z0-9- (they may have a "[LOCALE]" postfix)
~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop: error: required key "Name" in group "Desktop Entry" is not present
Error on file "~/rpmbuild/BUILDROOT/guake-0.6.0-1.fc23.x86_64/usr/share/applications/guake.desktop": Failed to validate the created desktop file
error: Bad exit status from /var/tmp/rpm-tmp.D5zGkS (%install)

So apparently efc129b made the desktop file invalid

@pypingou
Copy link
Member Author

Note: this is valid for both desktop files

@pypingou
Copy link
Member Author

This also means we actually lost translations in the desktop file:

0.6.0:

[Desktop Entry]
Encoding=UTF-8
Name=Guake Terminal
Comment=Use the command line in a Quake-like terminal
TryExec=guake
Exec=guake
Icon=guake
Type=Application
Categories=GNOME;GTK;System;Utility;TerminalEmulator;
StartupNotify=true
Keywords=Terminal;Utility;
X-Desktop-File-Install-Version=0.22

Previously

[Desktop Entry]
Encoding=UTF-8
Name=Guake Terminal
Name[pt]=Guake Terminal
Name[pt_BR]=Guake Terminal
Name[fr]=Guake Terminal
Name[fr_FR]=Guake Terminal
Comment=Use the command line in a Quake-like terminal
Comment[pt]=Utilizar a linha de comando em um terminal estilo Quake
Comment[pt_BR]=Utilizar a linha de comando em um terminal estilo Quake
Comment[fr]=Utilisez la ligne de commande comme dans un terminal quake
Comment[fr_FR]=Utilisez la ligne de commande comme dans un terminal quake
TryExec=guake
Exec=guake
Icon=guake
Type=Application
Categories=GNOME;GTK;System;Utility;TerminalEmulator;
StartupNotify=true
X-Desktop-File-Install-Version=0.15

gsemet added a commit that referenced this issue Apr 19, 2015
See #500 for information

Revert of efc129b

Signed-off-by: Gaetan Semet <[email protected]>
@piotrdrag
Copy link
Contributor

@gsemet
Copy link
Member

gsemet commented Apr 21, 2015

closed in favor of #501

@stouf
Copy link

stouf commented May 7, 2015

The Gnome .desktop files made invalid in efc129b are still in the package currently on ArchLinux mirrors (https://www.archlinux.org/packages/community/x86_64/guake/).
I don't know who is responsible for the packaging for the multiple Linux distributions, but just wanted to let you guys know about the issue.

gsemet added a commit that referenced this issue Jan 30, 2017
See #500 for information

Revert of efc129b

Signed-off-by: Gaetan Semet <[email protected]>
gsemet added a commit that referenced this issue Jan 30, 2017
See #500 for information

Revert of efc129b

Signed-off-by: Gaetan Semet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants