Ubuntu
quassel package

Wrong dispay in Plasma Indicator Display when quassel-client is installed but not quassel

Bug #502409 reported by Georges Dubus
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
quassel (Ubuntu)
Fix Released
Medium
Quassel

Bug Description

Binary package hint: quassel

When quassel-client is installed but not quassel, the Plasma Indicator displays "quassel.desktop" instead of the name and the icon of quassel.

This is due to the plasmoid trying to get information from /usr/share/applications/kde4/quassel.desktop whereas it's in the package quassel, not quassel-client.

I worked it around with a dirty hack : symlinking /usr/share/applications/kde4/quassel.desktop to /usr/share/applications/kde4/quasselclient.desktop

I don't know the how the plasmoid work, but It seems that quassel provides the wrong desktop file to the plasmoid.

Tags: notification
Georges Dubus (madjar)
tags: added: notification
Revision history for this message
Scott Kitterman (kitterman) wrote :

No, I think the indicator has to consider this case.

affects: quassel (Ubuntu) → libindicate-qt (Ubuntu)
Revision history for this message
Dave Walker (dogatemycomputer) wrote :

Thanks for reporting this bug and any supporting documentation. Since this bug has enough information provided for a developer to begin work, I'm going to mark it as confirmed and let them handle it from here. Thanks for taking the time to make Ubuntu better!

Changed in libindicate-qt (Ubuntu):
status: New → Confirmed
Revision history for this message
Georges Dubus (madjar) wrote :

I think I've found where the bug is : in the CMakeLists.txt file, line 297 ( http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/lucid/quassel/lucid/annotate/head:/CMakeLists.txt#L297 ). If I understand how it works, the compilation of the monolithic and client-only binaries is done at once : WANT_QTCLIENT and WANT_MONO both are on, so DESKTOP_FILE is set to quassel.desktop.

This is definitively a bug in quassel (or the way it's package in ubuntu to be precise). I see two ways to solve it :
- Change the way the notification are handled inside the quassel code, in order to have the client-only give the right quassel.desktop.
- Create a dummy .desktop file that would be in the quassel-data package and would only be used regarding to the notification.

What do you think ?

Revision history for this message
Scott Kitterman (kitterman) wrote :

The developer for libqtindicate-qt did the integration patches for quassel, so I'll leave it to him to decide.

Aurélien Gâteau (agateau)
Changed in libindicate-qt (Ubuntu):
assignee: nobody → Aurélien Gâteau (agateau)
Revision history for this message
Aurélien Gâteau (agateau) wrote :

This should fix the bug. Patch is against Quassel master, but it should apply to latest release as well.

Changed in libindicate-qt (Ubuntu):
importance: Undecided → Medium
status: Confirmed → In Progress
affects: libindicate-qt (Ubuntu) → quassel (Ubuntu)
Revision history for this message
Aurélien Gâteau (agateau) wrote :

Fix has been integrated in Quassel master

Changed in quassel (Ubuntu):
status: In Progress → Fix Committed
assignee: Aurélien Gâteau (agateau) → Quassel (quassel)
Revision history for this message
Georges Dubus (madjar) wrote :

Merci Aurélien

Scott Kitterman (kitterman)
Changed in quassel (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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