Comment 50 for bug 278471

Revision history for this message
Jakob Petsovits (jpetso) wrote :

You know, I'm really pissed. That KRandR kded module (aka "Detecting RANDR (monitor) changes") is *not* in KDE proper but only in an obviously unfinished working branch of the KWin lead developer, Lubos Lunak. See http://websvn.kde.org/branches/work/~seli/randr/module/?revision=864812 for the current location and contents of that code.

Congrats to Kubuntu for including unreleased code that explicitely specifies critical sections as "HACK", well done. The last commit (6 weeks ago) even disabled that timer, quote:

(...)
#if 0 // xrandr apparently can't detect hw changes and on some systems polling freezes X :(
    QTimer* timer = new QTimer( this );
    timer->start( 10000 ); // 10 s
    connect( timer, SIGNAL( timeout()), this, SLOT( poll()));
#endif
(...)

I suggest you people update the package to current SVN and leave it with that for Intrepid, so that not more damage is done than has already been inflicted. It won't work for events that happen during a KDE session then, but at least it doesn't break lots of systems in various subtle and annoying ways. Guess there's a reason that Lubos didn't include this into standard KDE packages yet.