Bug 202214 - Restart or logout window appear for each required restart or logout
Summary: Restart or logout window appear for each required restart or logout
Status: RESOLVED FIXED
Alias: None
Product: kpackagekit
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Fedora RPMs Unspecified
: NOR normal
Target Milestone: ---
Assignee: Steven M. Parrish
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-01 17:09 UTC by Grégoire Paris
Modified: 2009-08-17 22:08 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshot showung the bug (78.13 KB, image/png)
2009-08-01 17:10 UTC, Grégoire Paris
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Grégoire Paris 2009-08-01 17:09:24 UTC
Version:            (using KDE 4.2.4)
Installed from:    Fedora RPMs

If 3 packages require a system restart, and 4 other require a logout, 7 notification window appear on top of each other.
I would rather expect one or two windows with a summary.
Comment 1 Grégoire Paris 2009-08-01 17:10:31 UTC
Created attachment 35772 [details]
Screenshot showung the bug
Comment 2 Daniel Nicoletti 2009-08-03 14:48:55 UTC
Which version of KPackageKit are you using?
The correct behavior was to only show a different notification *IF* it's more important, like first was "you need to logout" then "you need to restart" in this case you would have two..
I tested that a lot but you version might not have the fix..
Comment 3 Tarjei Knapstad 2009-08-05 09:32:55 UTC
I'm having the exact same problem on Fedora 11 which I think has the latest version of kpackagekit, 0.4.1.1
Comment 4 Ali 2009-08-16 12:37:23 UTC
When an update begins, before finishing downloading and committing changes, numerous notifications to restart or logout appear. I think this a dangerous behavior for someone not well attentionned that can lead to a failed update with consequences on the system stability.

My system is a fedora 11 with gcc version 4.4.0, kpackagekit version 0.4.1.1, Kde 4.2.4
Comment 5 Daniel Nicoletti 2009-08-16 18:23:21 UTC
Sorry, I thought I had this fix into 0.4.1 series, but it wasn't even commited...
it's fixed in 0.4.2, but the various popups is Fedora backend problem.
Comment 6 Oliver Henshaw 2009-08-16 20:12:10 UTC
Should you also return early if restart type is RestartNone or UnknownRestartType? Otherwise I think you leak a KNotification.

Though I think in the first case you should never get that far but the second case can happen.

I think it would happen with PackageKit master, since at a glance it looks like  packagekit-qt has no equivalents of the new PK_RESTART_ENUM_SECURITY_SESSION or PK_RESTART_ENUM_SECURITY_SYSTEM restart types. But I don't know when packagekit will start using them.
Comment 7 Grégoire Paris 2009-08-17 19:53:48 UTC
(In reply to comment #5)
> Sorry, I thought I had this fix into 0.4.1 series, but it wasn't even
> commited...
> it's fixed in 0.4.2, but the various popups is Fedora backend problem.

Sorry, but what do you mean by "Fedora backend problem"? Should we do a bug report somewhere else? If yes, where?
Comment 8 Daniel Nicoletti 2009-08-17 20:32:46 UTC
Hmm I think a bug against PackageKit in Fedora might do it, the problem is that the yum backend is the one resposible for emiting a restartSignal, KPackageKit just show it, and I don't think KPK should hold them till the end of the transaction, yum backend is the one that should do it. The where is probably something like bugs.fedora.org (sorry if this address is wrong but i don't use fedora).
Comment 9 Grégoire Paris 2009-08-17 22:08:01 UTC
(In reply to comment #8)
> Hmm I think a bug against PackageKit in Fedora might do it, the problem is that
> the yum backend is the one resposible for emiting a restartSignal, KPackageKit
> just show it, and I don't think KPK should hold them till the end of the
> transaction, yum backend is the one that should do it. The where is probably
> something like bugs.fedora.org (sorry if this address is wrong but i don't use
> fedora).

Thanks for your reply, I reported the bug here : https://bugzilla.redhat.com/show_bug.cgi?id=517907 . Wait and see.