/usr/share/applications/qemu.desktop should have an "Exec=" key.

Bug #1868221 reported by Lockywolf
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
QEMU
Expired
Undecided
Unassigned

Bug Description

According to the www.freedesktop.org .desktop-file specification, all "Application" desktop files should have an "Exec=" key. The one in qemu doesn't.

This can be easily verified by running kbuildsycoca4 if KDE4 is present, but the issue is not DE-dependent.

Which binary exactly should be assigned as the default one, I don't know.

Tags: gui usability
Revision history for this message
Lockywolf (lockywolf) wrote :
Revision history for this message
Daniel Berrange (berrange) wrote :

Adding an exec field would not be right, because QEMU can't simply be launched from the desktop without any arguments. There needs to be a long string of arguments given that are different for every QEMU that is launched. The only viable way to actually launch QEMU is interactively from the terminal, or indirectly via a 3rd party app like virt-manager. We only created the qemu.desktop file in the first place because Wayland needs this to be present in order to identify what Icon to display for a window.

Revision history for this message
Daniel Berrange (berrange) wrote :

Note that QEMU sets the NoDisplay=true property to tell desktops not to display this entry. I don't think KDE should be warning about missing Exec entry in this case.

Revision history for this message
Lockywolf (lockywolf) wrote :

I'll report a bug in KDE and let's see if the guys agree. Maybe it is a deficiency of the .desktop specification.

Revision history for this message
Frédéric Brière (fbriere) wrote :

Thank you Lockywolf for this bug report. Have you filed one against KDE as you previously mentioned? If so, could you provide us with a link? Thanks in advance!

Revision history for this message
Lockywolf (lockywolf) wrote :

I am sorry I haven't dealt with this bug for quite a while. KDE 5 is not properly working on my distro, and I wanted to test it when it stabilises.

If qemu dislikes long-standing bugs, this bug can be closed, and I'll open a new one when I have time to test it on the new KDE.

Revision history for this message
Michael Tokarev (mjt+launchpad-tls) wrote :

What's the actual problem we're trying to solve here? What needs to be tested?

Revision history for this message
Lockywolf (lockywolf) wrote :

I can confirm that this behaviour is still present on kde 5.20.4.

You can run e.g. khelpcenter and observe:

kf.service.services: The desktop entry file "/usr/share/applications/qemu.desktop" has Type= "Application" but no Exec line
kf.service.sycoca: Invalid Service : "/usr/share/applications/qemu.desktop"

Revision history for this message
Lockywolf (lockywolf) wrote :
Revision history for this message
Thomas Huth (th-huth) wrote :

The QEMU project is currently considering to move its bug tracking to
another system. For this we need to know which bugs are still valid
and which could be closed already. Thus we are setting older bugs to
"Incomplete" now.

If you still think this bug report here is valid, then please switch
the state back to "New" within the next 60 days, otherwise this report
will be marked as "Expired". Or please mark it as "Fix Released" if
the problem has been solved with a newer version of QEMU already.

Thank you and sorry for the inconvenience.

Changed in qemu:
status: New → Incomplete
Lockywolf (lockywolf)
Changed in qemu:
status: Incomplete → New
Revision history for this message
Thomas Huth (th-huth) wrote : Moved bug report

This is an automated cleanup. This bug report has been moved to QEMU's
new bug tracker on gitlab.com and thus gets marked as 'expired' now.
Please continue with the discussion here:

 https://gitlab.com/qemu-project/qemu/-/issues/440

Changed in qemu:
status: New → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.