After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 786079 - general: Remove spaces from .desktop MimeType list
general: Remove spaces from .desktop MimeType list
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
3.26.x
Other All
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-08-10 00:56 UTC by Jeremy Bicha
Modified: 2017-08-10 07:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
general: Remove spaces from .desktop MimeType list (1.75 KB, patch)
2017-08-10 00:56 UTC, Jeremy Bicha
committed Details | Review

Description Jeremy Bicha 2017-08-10 00:56:46 UTC
nautilus 3.25.90
self-built on Ubuntu 17.10 Alpha

$ sudo update-desktop-database 
Error in file "/usr/share/applications/org.gnome.Nautilus.desktop": " application/x-7z-compressed" is an invalid MIME type (" application" is an unregistered media type)
Error in file "/usr/share/applications/org.gnome.Nautilus.desktop": " application/x-7z-compressed-tar" is an invalid MIME type (" application" is an unregistered media type)

etc.

Removing the spaces from the MimeType lines fixes this. Patch attached.
Comment 1 Jeremy Bicha 2017-08-10 00:56:50 UTC
Created attachment 357315 [details] [review]
general: Remove spaces from .desktop MimeType list
Comment 2 Carlos Soriano 2017-08-10 07:09:49 UTC
Review of attachment 357315 [details] [review]:

Hey,

Although I really don't mind having them or not, it didn't fail for neither of us (me and Ernestas), and we regenerate them in every build (see https://git.gnome.org/browse/nautilus/tree/build-aux/meson/postinstall.py), do you have a clue what could be the reason?
Comment 3 Jeremy Bicha 2017-08-10 07:36:43 UTC
Try running update-desktop-database without the -q option (quiet)
Comment 4 Carlos Soriano 2017-08-10 07:45:40 UTC
Review of attachment 357315 [details] [review]:

Figured it out, that script is not run under JHBuild, and it indeed raises errors otherwise.
Comment 5 Jeremy Bicha 2017-08-10 07:51:15 UTC
or you can run  desktop-file-validate

Attachment 357315 [details] pushed as 501ece6 - general: Remove spaces from .desktop MimeType list