201498 2007-12-06 17:51 0000 make repoman run desktop-file-validate on .desktop files 2007-12-30 17:31:01 0000 1 1 1 Unclassified Portage Development Repoman 2.2 All Linux RESOLVED FIXED http://dev.gentoo.org/~betelgeuse/desktop-files/baddesktop InSVN P2 normal --- 200044 1 betelgeuse@gentoo.org dev-portage@gentoo.org coldwind@gentoo.org ulm@gentoo.org betelgeuse@gentoo.org 2007-12-06 17:51:49 0000 We have lots of .desktop files in files/ so we should make repoman run desktop-file-validate on them if it's installed. Perhaps add a qa use flag to pull it in. Here are the current results: http://dev.gentoo.org/~betelgeuse/desktop-files/results.txt The script used to create these: http://dev.gentoo.org/~betelgeuse/desktop-files/baddesktop zmedico@gentoo.org 2007-12-06 18:04:42 0000 Is there any significant benefit to distributing a pre-generated .desktop file vs. generating one on-the-fly using make_desktop_entry from eutils.eclass? If not, perhaps we should just have repoman complain about any .desktop files in ${FILESDIR}. If there's something that makes make_desktop_entry() insufficient in lots of cases, maybe we can fix those shortcomings. betelgeuse@gentoo.org 2007-12-06 18:14:52 0000 (In reply to comment #1) > Is there any significant benefit to distributing a pre-generated .desktop file > vs. generating one on-the-fly using make_desktop_entry from eutils.eclass? If > not, perhaps we should just have repoman complain about any .desktop files in > ${FILESDIR}. > At least translations come to mind. ulm@gentoo.org 2007-12-07 00:33:47 0000 Apart from translations, "make_desktop_entry" supports only a small subset of the desktop entry keys mentioned in <http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s05.html>. My personal statistics is not very large, but in 3 of 3 cases I couldn't use make_desktop_entry because it was not flexible enough: - no support for a "Comment" different from $DESCRIPTION (needed because the ebuild installs two desktop files), - no support for "MimeType", "NoDisplay", and "Terminal" keys. Also, in case of "Exec" being anything other than a simple command (i.e. if it is a full path or has arguments), the generated file has a strange name. For example, "_usr_bin_emacsclient_-a__usr_bin_emacs_%F-emacs-desktop.desktop". (In reply to comment #1) > [...] perhaps we should just have repoman complain about any .desktop files > in {FILESDIR}. IMHO, that's not a good idea. vapier@gentoo.org 2007-12-07 06:44:49 0000 then why dont you extend the desktop creation function coldwind@gentoo.org 2007-12-10 19:34:45 0000 make_desktop_entry (or a new function) could take as an argument the shipped .desktop, fix some fields and add the ones which are missing. It could also print an ewarn when it find an invalid category. betelgeuse@gentoo.org 2007-12-23 02:46:19 0000 (In reply to comment #5) > make_desktop_entry (or a new function) could take as an argument the shipped > .desktop, fix some fields and add the ones which are missing. It could also > print an ewarn when it find an invalid category. > We already have domenu. coldwind@gentoo.org 2007-12-25 14:36:42 0000 (In reply to comment #6) > (In reply to comment #5) > > make_desktop_entry (or a new function) could take as an argument the shipped > > .desktop, fix some fields and add the ones which are missing. It could also > > print an ewarn when it find an invalid category. > > > > We already have domenu. > Fixing some fields, adding new ones or print warnings is still applicable to domenu. zmedico@gentoo.org 2007-12-30 17:31:01 0000 This has been released in 2.1.4_rc12.