I recently noticed that when I uninstalled dev-qt/assistant-4.x that I no longer had a working menu item in KDE for assistant despite still have 5.x installed. After a little bit of investigation, I've noticed this difference: the 4.x ebuilds have the following: multilib_src_install_all() { qt4_multilib_src_install_all doicon tools/assistant/tools/assistant/images/assistant.png make_desktop_entry assistant Assistant assistant 'Qt;Development;Documentation' } while the 5.x ebuilds do not. It would be nice if the ebuild create desktop entries as well so I don't have to manually create launchers for it. Reproducible: Always
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/qt.git/commit/?id=03235d3408db759181a4c82263cd1dd782e8abab commit 03235d3408db759181a4c82263cd1dd782e8abab Author: Michael Palimaka <kensington@gentoo.org> AuthorDate: 2017-12-25 02:42:02 +0000 Commit: Michael Palimaka <kensington@gentoo.org> CommitDate: 2017-12-25 02:42:14 +0000 dev-qt/assistant: install icons and desktop file Bug: https://bugs.gentoo.org/634566 Package-Manager: Portage-2.3.14, Repoman-2.3.6 dev-qt/assistant/assistant-5.10.0.ebuild | 20 +++++++++++++++++++- dev-qt/assistant/assistant-5.10.9999.ebuild | 20 +++++++++++++++++++- dev-qt/assistant/assistant-5.9.9999.ebuild | 20 +++++++++++++++++++- dev-qt/assistant/assistant-5.9999.ebuild | 20 +++++++++++++++++++- 4 files changed, 76 insertions(+), 4 deletions(-)}
Not just assistant... as far as I know, no qt5 tools install a desktop file (yet). In qt4, linguist, designer, and qdbusviewer also installed a desktop file. There may be more in qt5. The desktop file should work independently from qtchooser configuration, therefore the Exec= line should point directly to "${QT5_BINDIR}/${tool}". I would also include "Qt5" in the Name= line, to avoid ambiguities.
(In reply to Davide Pesavento from comment #2) > Not just assistant... as far as I know, no qt5 tools install a desktop file > (yet). > In qt4, linguist, designer, and qdbusviewer also installed a desktop file. > There may be more in qt5. Yep, there's definitely multiple, see also bug #577680. I just happened to pick this random bug to work on. > The desktop file should work independently from qtchooser configuration, > therefore the Exec= line should point directly to "${QT5_BINDIR}/${tool}". Thanks, fixed. > I would also include "Qt5" in the Name= line, to avoid ambiguities. Do you think it's worth it this close to Qt 4's demise? Of course, then this might future-proof us for Qt 6.
(In reply to Michael Palimaka (kensington) from comment #3) > (In reply to Davide Pesavento from comment #2) > > I would also include "Qt5" in the Name= line, to avoid ambiguities. > Do you think it's worth it this close to Qt 4's demise? Of course, then this > might future-proof us for Qt 6. Yes, not only for future-proofing, but also because having a menu entry that just says "Assistant" may not be very meaningful to the user.
For the record, this got updated to 'Qt 5 Assistant' in https://gitweb.gentoo.org/proj/qt.git/commit/?id=18b5775c3c9b1454e84f42c899ddddd7a766b2a8.
Fixed in 5.9.4.
