Summary: | [Tracker] Many KDE ebuilds fail desktop entry validity check | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Martin von Gagern <Martin.vGagern> |
Component: | [OLD] KDE | Assignee: | Gentoo KDE team <kde> |
Status: | RESOLVED WONTFIX | ||
Severity: | minor | CC: | esigra |
Priority: | High | Keywords: | Tracker |
Version: | 2007.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | 186942, 187371, 188755, 189551, 190006, 190758, 210057 | ||
Bug Blocks: | 245954 | ||
Attachments: |
desktop validation errors for 61 packages
remaining packages |
Description
Martin von Gagern
2007-08-18 13:47:27 UTC
As you made this a Tracker and have sub-bugs for individual ebuilds, I matched the errors I can find on my system to the corresponding ebuilds. Do you want to open a bug report for each of these? Or only packages outside kde-base? kde-base/ark-3.5.7 (ark.desktop) kde-base/certmanager-3.5.7-r1 (kleopatra_import.desktop) kde-base/kaboodle-3.5.7 (kaboodle.desktop) kde-base/kaddressbook-3.5.7 (kaddressbook.desktop) kde-base/kate-3.5.7-r1 (kate.desktop, kwrite.desktop) kde-base/kcontrol-3.5.7-r1 (installktheme.desktop, kfontview.desktop) kde-base/kdelirc-3.5.7 (irkick.desktop) kde-base/kdemultimedia-arts-3.5.7 (artsbuilder.desktop) kde-base/kdeprint-3.5.7 (printers.desktop) kde-base/kedit-3.5.7 (KEdit.desktop) kde-base/keduca-3.5.7 (keducabuilder.desktop, keduca.desktop) kde-base/kfax-3.5.7 (kfaxview.desktop) kde-base/kghostview-3.5.7 (kghostview.desktop) kde-base/kgpg-3.5.7 (kgpg.desktop) kde-base/kimagemapeditor-3.5.7 (kimagemapeditor.desktop) kde-base/klaptopdaemon-3.5.7-r1 (laptop.desktop) kde-base/kmail-3.5.7-r2 (kmail_view.desktop) kde-base/kmid-3.5.7 (kmid.desktop) kde-base/kmplot-3.5.7 (kmplot.desktop) kde-base/kolf-3.5.7 (kolf.desktop) kde-base/kolourpaint-3.5.7 (kolourpaint.desktop) kde-base/kommander-3.5.7 (kmdr-editor.desktop) kde-base/konqueror-3.5.7-r2 (kfmclient_dir.desktop, kfmclient_war.desktop) kde-base/konsole-3.5.7 (konsolesu.desktop) kde-base/kopete-3.5.7-r1 (kopete.desktop) kde-base/kpdf-3.5.7-r1 (kpdf.desktop) kde-base/kpoker-3.5.7 (kpoker.desktop) kde-base/ksysguard-3.5.7 (ksysguard.desktop) kde-base/ktouch-3.5.7 (ktouch.desktop) kde-base/ktuberling-3.5.7 (ktuberling.desktop) kde-base/kuickshow-3.5.5 (kuickshow.desktop) kde-base/kview-3.5.7 (kview.desktop) kde-base/kwalletmanager-3.5.7 (kwalletmanager.desktop) kde-base/noatun-3.5.7 (noatun.desktop) kde-base/quanta-3.5.7 (quanta.desktop) kde-base/superkaramba-3.5.7 (superkaramba.desktop) kde-base/umbrello-3.5.7 (umbrello.desktop) Martin, please only file individual bugs for packages outside of kde-base. Created attachment 129050 [details] desktop validation errors for 61 packages Now I ran this command on my system: locate \*.desktop | grep -vE '^/usr/portage|^/usr/local' | while read i; do desktop-file-validate $i done | grep error | cut -d: -f1 | qfile -v -f - | cut -d\ -f1 | sort -u to find out which other desktop files on my system are broken. I found KOffice, for which I filed bug 190006, and 61 more packages outside kde-base. I don't know which of these ebuilds run validate_desktop_entries, but whether or not, a bad desktop entry is a bad thing in any case, right? Right now I don't have the time to file 61 bug reports, especially since I can't specify a block with the initial submission, so I'd have to edit each report twice. Also I wouldn't consider my system representative, so perhaps the above command could be run on one of the tinderboxes or some other host with a large collection of built packages? I'm attaching the list of errors, in case someone is curious or willing to file a bunch of bugs for them, maybe using some bulk submission tool. Thanks, Martin! Not filing individual bugs for all these is better anyway. I simply didn't expect that many. We'll look at your list instead. Do you plan to provide some desktop entry fixing function in some eclass? In this case it would be nice to let Thomas A. know about it, as he's preparing fixes to individual ebuilds in bug 190006. Missing semicolon can be fixed easily, and forbidden keys kan be removed or prefixed with X-. To fix the Categories, the ebuild must probably provide the correct category, but the actual text replacement can be in some eclass as well. The same goes for missing names. Incorrect UTF-8 format can be handled by finding these sequences, assuming they are ISO-8859-1, and recoding them. This Version business I haven't understood yet, will probably have to read the sources of the checker or some desktop file format spec. I guess I would want an eclass for desktop entry mangling, inherited by kde eclass and some individual ebuilds. I would want a separate function for each fix, such that the ebuild still has to say what's wrong, but in a readable way, so that you know what it is fixing, not so much how it is fixing it. You still could have some auto-fix function calling the individual ones if absolutely needed. In this case that function should probably be versioned somehow, so that you can write one with different corrections in the future without modifying the behavior of current ebuilds. Created attachment 192506 [details]
remaining packages
KDE 3.5 is going away soon, this won't be fixed |