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 696489 - Add test to validate .desktop files
Add test to validate .desktop files
Status: RESOLVED OBSOLETE
Product: gnome-continuous
Classification: Other
Component: general
unspecified
Other All
: Normal enhancement
: ---
Assigned To: GNOME Continuous maintainer(s)
GNOME Continuous maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-03-24 12:07 UTC by Philip Withnall
Modified: 2021-06-05 16:29 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Philip Withnall 2013-03-24 12:07:30 UTC
As discussed on gnome-i18n[1], it would be useful if there was a pre-receive hook to prevent translators pushing commits which break .desktop files. This hook could use desktop-file-validate[2] to check the .desktop files; similarly to how the existing pre-receive-check-po script uses msgfmt to check .po files.

[1]: https://mail.gnome.org/archives/gnome-i18n/2013-March/msg00211.html
[2]: http://freedesktop.org/wiki/Software/desktop-file-utils
Comment 1 Daniel Mustieles 2013-03-24 12:17:17 UTC
It also would be useful to check documnetation syntax with gtxml (https://launchpad.net/pyg3t) to avoid typos in documentation tags.
Comment 2 Matthias Clasen 2013-03-24 20:47:56 UTC
Two problems here:

- the desktop files we need to check are generated files, they don't actually exist in git. You'd have to build to get them

- the desktop-file-validate has to be reasonably new. The version that is in rhel5 or rhel6 almost certainly won't do
Comment 3 Philip Withnall 2013-03-24 21:07:00 UTC
(In reply to comment #2)
> - the desktop files we need to check are generated files, they don't actually
> exist in git. You'd have to build to get them

Hmm, that’s a pain. It might be possible to run intltool-merge on the git tree and generate the .desktop files in a temporary directory from the hook; i.e. run the @INTLTOOL_DESKTOP_RULE@ code in the hook.
Comment 4 Andrea Veri 2013-11-21 14:57:19 UTC
The GNOME Infrastructure Team is currently migrating its bug / issue tracker away from Bugzilla to Request Tracker and therefore all the currently open bugs have been closed and marked as OBSOLETE.

The following move will also act as a cleanup for very old and ancient tickets that were still living on Bugzilla. If your issue still hasn't been fixed as of today please report it again on the relevant RT queue.

More details about the available queues you can report the bug against can be found at https://wiki.gnome.org/Sysadmin/RequestTracker.

Thanks for your patience,

the GNOME Infrastructure Team
Comment 5 André Klapper 2014-05-18 11:45:36 UTC
https://rt.gnome.org/Ticket/Display.html?id=14101 now
Comment 6 André Klapper 2014-05-18 11:45:39 UTC
*** Bug 723138 has been marked as a duplicate of this bug. ***
Comment 7 Olav Vitters 2014-07-29 09:03:25 UTC
I added a imperfect pre-commit hook to git.gnome.org as part of bug 723138 (unduped it). I think this is a perfect thing that could be integrated with gnome-continuous.
Comment 8 André Klapper 2021-06-05 16:29:52 UTC
gnome-continuous is not under active development anymore. Its codebase has been archived: https://gitlab.gnome.org/Archive/gnome-continuous

Closing all its open tickets as part of housekeeping.