FS#42385 - [kdelibs] [kinit] Autostart hangs in kinit

Attached to Project: Arch Linux
Opened by AnAkkk (AnAkkk) - Wednesday, 15 October 2014, 09:23 GMT
Last edited by Andrea Scarpino (BaSh) - Tuesday, 28 October 2014, 18:42 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Andrea Scarpino (BaSh)
Sven-Hendrik Haase (Svenstaro)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

This is an old bug, it exists in both KDE4 and KF5. I have it on two computers with KF5, it causes various issues (plasma shell not launching, or shutdown/logout buttons not working).

It has been reported a long time ago but isn't fixed:
https://bugs.kde.org/show_bug.cgi?id=328571

There is a workaround I am using though, I've attached the patch to the bug report. Could it be added to the kinit package?
   init_fix.patch (0.7 KiB)
This task depends upon

Closed by  Andrea Scarpino (BaSh)
Tuesday, 28 October 2014, 18:42 GMT
Reason for closing:  Upstream
Additional comments about closing:  Please press on upstream to apply the patch.
Comment by zless (roentgen) - Wednesday, 15 October 2014, 17:57 GMT
Quote from the linked bug report:

"And I can't approve a patch that works around an issue without a proper explanation of what the issue is. "Never commit code you don't understand" is a rather universal rule of software development."
Comment by AnAkkk (AnAkkk) - Wednesday, 15 October 2014, 17:59 GMT
So it's better to leave it broken? Fedora already uses a similar patch.
Comment by patrick (potomac) - Thursday, 16 October 2014, 02:21 GMT
it seems dangerous to use a code that you don't understand, the patch can fix your problem but it can also create new problems,

you can use makepkg in order to build your own version of kdelibs package, it's easy to do with makepkg
Comment by AnAkkk (AnAkkk) - Thursday, 16 October 2014, 09:12 GMT
There's nothing I don't understand about this patch. There's a bug in Glib event loop, the only way around it is to use Qt event loop. You do that with the env variable that the patch set. I don't think Fedora would have included a patch they don't understand, and kwin does the exact same thing (QT_NO_GLIB=1) on start due to bugs with the glib event loop.
Comment by patrick (potomac) - Thursday, 16 October 2014, 15:05 GMT
it seems better to wait the definitive solution from the KDE developers,

is this patch is the definitive solution for this bug ?
Comment by AnAkkk (AnAkkk) - Thursday, 16 October 2014, 16:11 GMT
Well, the bug report has been opened since December 2013, so we might still wait a long time before it's fixed. I guess that patch is better than leaving the users with a black screen or no shutdown/logout button.

Loading...