[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #59 from Nick Shaforostoff sha...@ukr.net --- yeah, generally kde5 became slower on my netbook. it takes 5 seconds for alt+f2 runner to start reacting to my input, it takes ~5 seconds for start menu and simple alt+tab switcher to appear, and it takes ~5-10 seconds just to show session-end/reboot/poweroff confirmation screen. -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #60 from Hrvoje Senjan hrvoje.sen...@gmail.com --- kded5 bt w/ Davids patch: #0 0x7f505a91685f in pthread_cond_wait@@GLIBC_2.3.2 () at /lib64/libpthread.so.0 #1 0x7f505b89796b in QWaitCondition::wait(QMutex*, unsigned long) (time=18446744073709551615, this=0xd11fe0) at thread/qwaitcondition_unix.cpp:128 #2 0x7f505b89796b in QWaitCondition::wait(QMutex*, unsigned long) (this=this@entry=0xd11fc8, mutex=mutex@entry=0xd11fa0, time=time@entry=18446744073709551615) at thread/qwaitcondition_unix.cpp:200 #3 0x7f505b89657e in QThread::wait(unsigned long) (this=this@entry= 0x7fffac915880, time=time@entry=18446744073709551615) at thread/qthread_unix.cpp:670 #4 0x7f502ad73462 in PowerDevilUPowerBackend::kjobExecInThread(KJob*) (job=job@entry=0xd95ed0) at /home/hrvoje/Src/local/powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp:594 #5 0x7f502ad73576 in PowerDevilUPowerBackend::brightnessValue(PowerDevil::BackendInterface::BrightnessControlType) const (this= 0xb6d580, type=optimized out) at /home/hrvoje/Src/local/powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp:308 #6 0x7f502ad710cd in PowerDevilUPowerBackend::init() (this=optimized out) at /home/hrvoje/Src/local/powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp:182 #7 0x7f502ab215db in PowerDevil::Core::loadCore(PowerDevil::BackendInterfac---Type return to continue, or q return to quit--- e*) (this=0xcb51a0, backend=backend@entry=0xb6d580) at /home/hrvoje/Src/local/powerdevil/daemon/powerdevilcore.cpp:87 #8 0x7f502ad649bc in KDEDPowerDevil::init() (this=0xc87670) at /home/hrvoje/Src/local/powerdevil/daemon/kdedpowerdevil.cpp:89 #9 0x7f505baa1506 in QObject::event(QEvent*) (this=0xc87670, e=optimized out) at kernel/qobject.cpp:1245 #10 0x7f505c72fb5c in QApplicationPrivate::notify_helper(QObject*, QEvent*) (this=this@entry=0x8a9bf0, receiver=receiver@entry=0xc87670, e=e@entry= 0xc74760) at kernel/qapplication.cpp:3722 #11 0x7f505c734bc0 in QApplication::notify(QObject*, QEvent*) (this= 0x7fffac916350, receiver=0xc87670, e=0xc74760) at kernel/qapplication.cpp:3505 #12 0x7f505ba70e55 in QCoreApplication::notifyInternal(QObject*, QEvent*) (this=0x7fffac916350, receiver=0xc87670, event=event@entry=0xc74760) at kernel/qcoreapplication.cpp:930 #13 0x7f505ba72cef in QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) (event=0xc74760, receiver=optimized out) at kernel/qcoreapplication.h:228 #14 0x7f505ba72cef in QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) (receiver=receiver@entry=0x0, event_type=event_type@entry=0, data=0x8a9d60) at kernel/qcoreapplication.cpp:1534 #15 0x7f505ba73328 in QCoreApplication::sendPostedEvents(QObject*, int) (receiver=receiver@entry=0x0, event_type=event_type@entry=0) ---Type return to continue, or q return to quit--- at kernel/qcoreapplication.cpp:1392 #16 0x7f505bac8d23 in postEventSourceDispatch(GSource*, GSourceFunc, gpointer) (s=0x91bed0) at kernel/qeventdispatcher_glib.cpp:271 #17 0x7f5059c97a04 in g_main_context_dispatch () at /usr/lib64/libglib-2.0.so.0 #18 0x7f5059c97c48 in () at /usr/lib64/libglib-2.0.so.0 #19 0x7f5059c97cec in g_main_context_iteration () at /usr/lib64/libglib-2.0.so.0 #20 0x7f505bac819c in QEventDispatcherGlib::processEvents(QFlagsQEventLoop::ProcessEventsFlag) (this=0x9105c0, flags=...) at kernel/qeventdispatcher_glib.cpp:418 #21 0x7f505ba6edab in QEventLoop::exec(QFlagsQEventLoop::ProcessEventsFlag) (this=this@entry=0x7fffac916280, flags=..., flags@entry=...) at kernel/qeventloop.cpp:204 #22 0x7f505ba76436 in QCoreApplication::exec() () at kernel/qcoreapplication.cpp:1183 #23 0x7f505dfcf73a in kdemain () at /usr/lib64/libkdeinit5_kded5.so #24 0x7f505dc48b45 in __libc_start_main () at /lib64/libc.so.6 #25 0x004007ee in _start () -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #61 from David Edmundson k...@davidedmundson.co.uk --- Thanks for testing. It seems the only solution will be to remove these blocking KAuth calls throughout this function. It's a wrong thing to be doing from kded. I suggest we query brightnessValueMax in init(); and use m_cachedBrightnessMap for both the getters. For the set case, we don't really need to wait for a reply. Is the error code really that useful? -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #58 from Hrvoje Senjan hrvoje.sen...@gmail.com --- from short testing the patch seems to resolves the CPU problem. however: 1) for locking the screen, it now takes some 30s to actually show the locker, 2) konsole action (right click) is blocked with: 0 0x7f6f482e94ad in poll () at /lib64/libc.so.6 #1 0x7f6f3bf1926a in () at /lib64/libdbus-1.so.3 #2 0x7f6f3bf180bf in () at /lib64/libdbus-1.so.3 #3 0x7f6f3bf025bc in () at /lib64/libdbus-1.so.3 #4 0x7f6f3bf02f69 in () at /lib64/libdbus-1.so.3 #5 0x7f6f3bf0352d in dbus_connection_send_with_reply_and_block () at /lib64/libdbus-1.so.3 #6 0x7f6f42666780 in QDBusConnectionPrivate::sendWithReply(QDBusMessage const, int, int) (error=0x7fff89162670, timeout_milliseconds=-1, message=0x2077aa0, connection=optimized out) at qdbus_symbols_p.h:135 #7 0x7f6f42666780 in QDBusConnectionPrivate::sendWithReply(QDBusMessage const, int, int) (this= 0x206b910, message=..., sendMode=optimized out, timeout=-1) at qdbusintegrator.cpp:2046 #8 0x7f6f426533c3 in QDBusConnection::call(QDBusMessage const, QDBus::CallMode, int) const (this=this@entry=0x23e2b10, message=..., mode=mode@entry=QDBus::Block, timeout=optimized out) at qdbusconnection.cpp:576 #9 0x7f6f426717ef in QDBusAbstractInterface::callWithArgumentList(QDBus::CallMode, QString const, QListQVariant const) (this=this@entry=0x7fff891629b0, mode=QDBus::Block, mode@entry=QDBus::AutoDetect, method=..., args=...) at qdbusabstractinterface.cpp:476 #10 0x7f6f426725b5 in QDBusAbstractInterface::call(QDBus::CallMode, QString const, QVariant const, QVariant const, QVariant const, QVariant const, QVar---Type return to continue, or q return to quit--- iant const, QVariant const, QVariant const, QVariant const) (this=this@entry=0x7fff891629b0, mode=mode@entry=QDBus::AutoDetect, method=..., arg1=..., arg2=..., arg3=..., arg4=..., arg5=..., arg6=..., arg7=..., arg8=...) at qdbusabstractinterface.cpp:746 #11 0x7f6f42672771 in QDBusAbstractInterface::call(QString const, QVariant const, QVariant const, QVariant const, QVariant const, QVariant const, QVariant const, QVariant const, QVariant const) (this=this@entry=0x7fff891629b0, method=..., arg1=..., arg2=..., arg3=..., arg4=..., arg5=..., arg6=..., arg7=..., arg8=...) at qdbusabstractinterface.cpp:689 #12 0x7f6f433c783c in KIO::favIconForUrl(QUrl const) (url=...) at /usr/src/debug/kio-5.7.0git/src/core/global.cpp:339 #13 0x7f6f433c8085 in KIO::iconNameForUrl(QUrl const) (url=...) at /usr/src/debug/kio-5.7.0git/src/core/global.cpp:356 #14 0x7f6f21034352 in SearchProvider::iconName() const (this=optimized out) at /usr/src/debug/kio-5.7.0git/src/urifilters/ikws/searchprovider.cpp:111 #15 0x7f6f4731313a in KUriFilterData::iconNameForPreferredSearchProvider(QString const) const (this=optimized out, provider=...) at /usr/src/debug/kio-5.7.0git/src/widgets/kurifilter.cpp:381 #16 0x7f6f47f83883 in Konsole::SessionController::updateWebSearchMenu() () at /usr/lib64/libkonsoleprivate.so.3 #17 0x7f6f47f89697 in Konsole::SessionController::showDisplayContextMenu(QPoint const) () at /usr/lib64/libkonsoleprivate.so.3 #18 0x7f6f4494e12f in QMetaObject::activate(QObject*, int, int, void**) (a=0---Type return to continue, or q return to quit--- x7fff89162eb0, r=0x2384940, this=0x2380270) at ../../src/corelib/kernel/qobject_impl.h:124 #19 0x7f6f4494e12f in QMetaObject::activate(QObject*, int, int, void**) (sender=0x23efd50, signalOffset=optimized out, local_signal_index=optimized out, argv=0x7fff89162eb0) at kernel/qobject.cpp:3702 #20 0x7f6f47fbfc55 in Konsole::TerminalDisplay::configureRequest(QPoint const) () at /usr/lib64/libkonsoleprivate.so.3 #21 0x7f6f47fa5269 in Konsole::TerminalDisplay::mousePressEvent(QMouseEvent*) () at /usr/lib64/libkonsoleprivate.so.3 #22 0x7f6f4561d50a in QWidget::event(QEvent*) (this=0x23efd50, event= 0x7fff891633b0) at kernel/qwidget.cpp:8652 #23 0x7f6f47fa5cca in Konsole::TerminalDisplay::event(QEvent*) () at /usr/lib64/libkonsoleprivate.so.3 #24 0x7f6f455ddb5c in QApplicationPrivate::notify_helper(QObject*, QEvent*) (this=this@entry=0x2094b20, receiver=receiver@entry=0x23efd50, e=e@entry=0x7fff891633b0) at kernel/qapplication.cpp:3722 #25 0x7f6f455e34e6 in QApplication::notify(QObject*, QEvent*) (this=optimized out, receiver=0x23efd50, e=0x7fff891633b0) at kernel/qapplication.cpp:3280 #26 0x7f6f4491ee55 in QCoreApplication::notifyInternal(QObject*, QEvent*) (this= 0x7fff89163cd0, receiver=receiver@entry=0x23efd50, event=event@entry=0x7fff891633b0) at kernel/qcoreapplication.cpp:930 #27 0x7f6f455e1eb1 in QApplicationPrivate::sendMouseEvent(QWidget*, QMouseEv---Type return to continue, or q return to quit--- ent*, QWidget*, QWidget*, QWidget**, QPointerQWidget, bool) (event=0x7fff891633b0, receiver=0x23efd50) at
[kwrited] [Bug 343532] New: empty notification string on wall messages
https://bugs.kde.org/show_bug.cgi?id=343532 Bug ID: 343532 Summary: empty notification string on wall messages Product: kwrited Version: unspecified Platform: Archlinux Packages URL: http://www.youtube.com/watch?v=9W_2cPsybnw OS: Linux Status: UNCONFIRMED Severity: minor Priority: NOR Component: general Assignee: plasma-devel@kde.org Reporter: ra...@abv.bg system messages don't show in notification tray popup Reproducible: Always Steps to Reproduce: 1. Open a terminal 2. Type wall some_random_message 3. Observe the notification tray popup with an empty string after : -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122312: Use sysconf() instead of relying on UT_NAMESIZE
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122312/ --- Review request for Plasma and Raphael Kubo da Costa. Repository: user-manager Description --- Use sysconf() instead of relying on UT_NAMESIZE This will work on any UNIX system even those that don't have utmp.h such as FreeBSD. Diffs - src/accountinfo.cpp 68fc865 Diff: https://git.reviewboard.kde.org/r/122312/diff/ Testing --- compiles, runs and shows user name too long tooltip when I enter a really long name Thanks, Alex Richardson ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[kwrited] [Bug 343532] empty notification string on wall messages
https://bugs.kde.org/show_bug.cgi?id=343532 Martin Klapetek mklape...@kde.org changed: What|Removed |Added CC||mklape...@kde.org --- Comment #1 from Martin Klapetek mklape...@kde.org --- wall doesn't do anything at all here, do I need to enable something somewhere? -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[plasmashell] [Bug 343532] empty notification string on wall messages
https://bugs.kde.org/show_bug.cgi?id=343532 Martin Klapetek mklape...@kde.org changed: What|Removed |Added Version|unspecified |master Assignee|plasma-devel@kde.org|mklape...@kde.org Component|general |Notifications Product|kwrited |plasmashell Target Milestone|--- |1.0 -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 120040: Install kdesu under bin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120040/ --- (Updated Jan. 29, 2015, 3:16 p.m.) Status -- This change has been discarded. Review request for KDE Frameworks and Plasma. Repository: kde-cli-tools Description --- this is a part of adressing the bug https://bugs.kde.org/show_bug.cgi?id=338755 https://bugs.kde.org/show_bug.cgi?id=338756 in kde4 kdesu was installed under bin, and should still, being something that should be invokable but renames it to kdesu5, for coinstallability reasons therefore, kio/src/core/desktopexecparser.cpp should be modified to search for kdesu5 instead of kdesu Diffs - kdesu/CMakeLists.txt 2a70831 Diff: https://git.reviewboard.kde.org/r/120040/diff/ Testing --- Thanks, Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122304: qtquickcontrols: Implement an ApplicationWindowStyle stub.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122304/#review74974 --- Ship it! Ship It! - Marco Martin On Jan. 29, 2015, 8:53 a.m., Nikita Skovoroda wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122304/ --- (Updated Jan. 29, 2015, 8:53 a.m.) Review request for Plasma, Andrew Lake and Marco Martin. Repository: breeze Description --- This commit adds a stub implementing ApplicationWindowStyle. This is required for qml applications to automatically use SystemPallete background color instead of a white one when launched with QT_QUICK_CONTROLS_STYLE=Breeze. Borders etc. are not altered, this just loads the default ApplicationWindowStyle implementation. Diffs - qtquickcontrols/ApplicationWindowStyle.qml PRE-CREATION qtquickcontrols/Breeze.qml df27579 qtquickcontrols/qmldir 685e763 Diff: https://git.reviewboard.kde.org/r/122304/diff/ Testing --- Works for me, fixes background for qml applications that are using ApplicationWindow{}. Thanks, Nikita Skovoroda ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122289: [applets/clipboard] Force tooltips to be PlainText
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122289/ --- (Updated Jan. 29, 2015, 8:07 a.m.) Status -- This change has been marked as submitted. Review request for Plasma and Martin Klapetek. Bugs: 343301 https://bugs.kde.org/show_bug.cgi?id=343301 Repository: plasma-workspace Description --- Prevents cross-side scripting attempts. This requires 8044e15 of plasma-framework. BUG: 343301 FIXED-IN: 5.3.0 Diffs - applets/clipboard/contents/ui/clipboard.qml 6ff1e2b56588293df76a24de92727a9f24437fdc Diff: https://git.reviewboard.kde.org/r/122289/diff/ Testing --- Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122304: qtquickcontrols: Implement an ApplicationWindowStyle stub.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122304/ --- Review request for Plasma. Repository: breeze Description --- This commit adds a stub implementing ApplicationWindowStyle. This is required for qml applications to automatically use SystemPallete background color instead of a white one when launched with QT_QUICK_CONTROLS_STYLE=Breeze. Borders etc. are not altered, this just loads the default ApplicationWindowStyle implementation. Diffs - qtquickcontrols/ApplicationWindowStyle.qml PRE-CREATION qtquickcontrols/Breeze.qml df27579 qtquickcontrols/qmldir 685e763 Diff: https://git.reviewboard.kde.org/r/122304/diff/ Testing --- Works for me, fixes background for qml applications that are using ApplicationWindow{}. Thanks, Nikita Skovoroda ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122304: qtquickcontrols: Implement an ApplicationWindowStyle stub.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122304/ --- (Updated Янв. 29, 2015, 8:53 д.п.) Review request for Plasma, Andrew Lake and Marco Martin. Repository: breeze Description --- This commit adds a stub implementing ApplicationWindowStyle. This is required for qml applications to automatically use SystemPallete background color instead of a white one when launched with QT_QUICK_CONTROLS_STYLE=Breeze. Borders etc. are not altered, this just loads the default ApplicationWindowStyle implementation. Diffs - qtquickcontrols/ApplicationWindowStyle.qml PRE-CREATION qtquickcontrols/Breeze.qml df27579 qtquickcontrols/qmldir 685e763 Diff: https://git.reviewboard.kde.org/r/122304/diff/ Testing --- Works for me, fixes background for qml applications that are using ApplicationWindow{}. Thanks, Nikita Skovoroda ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5.2 beta
It would be better, if you could verify these things in 5.2 stable. If you are able to reproduce them, file a bug report at https://bugs.kde.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #56 from Nick Shaforostoff sha...@ukr.net --- are you this patch would really matter, taking into account the fact that i have /usr/share/dbus-1/system-services/org.kde.powerdevil.backlighthelper.service pointing to existing executable, which is linked against kf5 libraries? -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
An appeal for continuing with KDE Activities
Hello Plasma Devs, Not sure if this is the right mailing list, but I've been seeing a lot on Planet KDE from people unsure of whether Activities are used and should be kept, etc. So I wrote this specifically in hopes of making sure the devs see how it's being used by regular folks. http://www.ericsbinaryworld.com/2015/01/27/an-appeal-for-keeping-kde-activities/ Thank you for developing an awesome desktop environment - I enjoy it so much that it makes me sad we don't use it at work. (Well, we'd first have to start using Linux...) ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Plasma 5.2 beta
Plasma 5.2 beta Hardware: Sony Vaio Pro 13, Inter i5, 4GB RAM, 128GB SSD 13' Touchscreen display @ 1920x1080 1. Machine hangs after screen is powered and PC goes to sleep Description: When the machine has been not been in use for 30 minutes, the screen turns off completely. Leaving the machine in this state for a lengthy periode resulted in the machine not being able to wake up again. The Lock screen is displayed as expected but it does not respond to any user interaction. The clock on the screen has also not been updated since the machine went to sleep. Remedy: Hard restart by holding the powere button for a few seconds. 2. Touchpad settings forgotten when machine wakes up from sleep Description: User logs in to machine with only the touchpad as pointing device. Touchpad settings are modified to taps are registered as clicks, and the top right is defined as right-click. An external USB mouse is connected later in the same session. After working, PC lid is closed. When the PC lid is later openned (mouse is now disconnected after commuting), and user logs in, the touchpad settings are no longer active and user has to open the system settings again. // // Aziz Mwondha // ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: An appeal for continuing with KDE Activities
Hi, Activity-specific power management settings will return in 5.3 :) Cheers, Kai Uwe ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122304: qtquickcontrols: Implement an ApplicationWindowStyle stub.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122304/ --- (Updated Jan. 29, 2015, 9:54 a.m.) Status -- This change has been marked as submitted. Review request for Plasma, Andrew Lake and Marco Martin. Repository: breeze Description --- This commit adds a stub implementing ApplicationWindowStyle. This is required for qml applications to automatically use SystemPallete background color instead of a white one when launched with QT_QUICK_CONTROLS_STYLE=Breeze. Borders etc. are not altered, this just loads the default ApplicationWindowStyle implementation. Diffs - qtquickcontrols/ApplicationWindowStyle.qml PRE-CREATION qtquickcontrols/Breeze.qml df27579 qtquickcontrols/qmldir 685e763 Diff: https://git.reviewboard.kde.org/r/122304/diff/ Testing --- Works for me, fixes background for qml applications that are using ApplicationWindow{}. Thanks, Nikita Skovoroda ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #55 from Kai Uwe Broulik k...@privat.broulik.de --- Does anyone of you have the possibility of reproducing this with an up to date KAuth, specifically this [1] patch? There have been attempts to fix the KJob fallback getting stuck but this patch will only be in Frameworks 5.7 which is not yet released. [1] https://git.reviewboard.kde.org/r/122029/ -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 --- Comment #57 from David Edmundson k...@davidedmundson.co.uk --- Can someone try this please https://git.reviewboard.kde.org/r/122307/ -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Proposal regarding plasmate's release
On Thu, Jan 29, 2015 at 1:36 PM, Giorgos Tsiapaliokas giorgos.tsiapalio...@kde.org wrote: Hello, I believe that before we release plasmate we should reconsider the release strategy. The current situation is that 1. the plasmate repo is in extragear 2. plasmate's repo version doesn't match the one of plasma. 3. the master branch contains the KDE4 code and the frameworks branch contains the KF5 code. I propose to 1. move the plasmate repo away from extragear For policy reasons this requires an email to kde-core-devel 2. release the plasmate repo together with plasma and all the applications in it will follow plasma's version number. So there won't be plasmoidviewer 2.0 but plasmoidviewer 5.x 3. merge the frameworks branch into master and delete the frameworks branch Can we do this merge before we do the actual moving. Otherwise we have to update the metadata twice. 4. make the PLASMATE_BUILD_WITH_KDEVPLATFORM off(false) by default and add a message that will inform the user that the plasmate application isn't ready yet for production. Other than that, to all the above. -- Giorgos Tsiapaliokas terietor.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Build failed in Jenkins: plasma-workspace_master_qt5 #1269
See http://build.kde.org/job/plasma-workspace_master_qt5/1269/changes Changes: [shafff] port kcminit away from kdelibs4support -- [...truncated 2376 lines...] [ 39%] Building CXX object klipper/CMakeFiles/kdeinit_klipper.dir/historyimageitem.cpp.o [ 39%] Building CXX object klipper/autotests/CMakeFiles/testHistoryModel.dir/__/historyimageitem.cpp.o [ 39%] Building CXX object klipper/CMakeFiles/plasma_engine_clipboard.dir/historymodel.cpp.o [ 39%] Building CXX object klipper/autotests/CMakeFiles/testHistoryModel.dir/__/historyitem.cpp.o [ 39%] Generating appadaptor.cpp, appadaptor.h [ 39%] Generating appadaptor.moc [ 39%] Building CXX object klipper/CMakeFiles/kdeinit_klipper.dir/historyurlitem.cpp.o Scanning dependencies of target krunner [ 39%] Building CXX object krunner/CMakeFiles/krunner.dir/main.cpp.o Linking CXX executable testHistory http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘bool TaskManager::Task::updateDemandsAttentionState(WId)’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:39:88: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations] NETWinInfo i(QX11Info::connection(), w, QX11Info::appRootWindow(), NET::WMState); ^ http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘int TaskManager::Task::pid() const’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:87:92: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations] return NETWinInfo(QX11Info::connection(), d-win, QX11Info::appRootWindow(), NET::WMPid).pid(); ^ http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘void TaskManager::Task::setMaximized(bool)’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:145:90: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations] NETWinInfo ni(QX11Info::connection(), d-win, QX11Info::appRootWindow(), NET::WMState); ^ http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘void TaskManager::Task::restore()’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:171:90: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations] NETWinInfo ni(QX11Info::connection(), d-win, QX11Info::appRootWindow(), NET::WMState); ^ http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘void TaskManager::Task::setAlwaysOnTop(bool)’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:207:90: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations] NETWinInfo ni(QX11Info::connection(), d-win, QX11Info::appRootWindow(), NET::WMState); ^ http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp: In member function ‘void TaskManager::Task::setKeptBelowOthers(bool)’: http://build.kde.org/job/plasma-workspace_master_qt5/ws/libtaskmanager/task_x11.cpp:216:90: warning: ‘NETWinInfo::NETWinInfo(xcb_connection_t*, xcb_window_t, xcb_window_t, NET::Properties, NET::Role)’ is deprecated (declared at /srv/jenkins/install/linux/x86_64/g++/kf5-qt5/frameworks/kwindowsystem/inst/include/KF5/KWindowSystem/netwm.h:988) [-Wdeprecated-declarations]
Proposal regarding plasmate's release
Hello, I believe that before we release plasmate we should reconsider the release strategy. The current situation is that 1. the plasmate repo is in extragear 2. plasmate's repo version doesn't match the one of plasma. 3. the master branch contains the KDE4 code and the frameworks branch contains the KF5 code. I propose to 1. move the plasmate repo away from extragear 2. release the plasmate repo together with plasma and all the applications in it will follow plasma's version number. So there won't be plasmoidviewer 2.0 but plasmoidviewer 5.x 3. merge the frameworks branch into master and delete the frameworks branch 4. make the PLASMATE_BUILD_WITH_KDEVPLATFORM off(false) by default and add a message that will inform the user that the plasmate application isn't ready yet for production. -- Giorgos Tsiapaliokas terietor.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Proposal regarding plasmate's release
On Thursday 29 January 2015 13:40:20 David Edmundson wrote: plasmoidviewer 2.0 but plasmoidviewer 5.x 3. merge the frameworks branch into master and delete the frameworks branch Can we do this merge before we do the actual moving. Otherwise we have to update the metadata twice. yeah, master should be in releasable state before moving. last thing, are there changes in master for kde4 not released? it may be good at least tagging it if so. -- Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Proposal regarding plasmate's release
These plans sound good to me. Thanks Giorgos for getting the ball rolling. On Thursday, January 29, 2015 14:36:13 Giorgos Tsiapaliokas wrote: 4. make the PLASMATE_BUILD_WITH_KDEVPLATFORM off(false) by default and add a message that will inform the user that the plasmate application isn't ready yet for production. Makes sense. Later on, when plasmate's new version is usable and useful, I'd like to turn this into an optional dependency, so if kdevplatform is there, we'll automatically build the plasmate IDE. Other than that, as said by others, +1 more. :) Cheers, -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: An appeal for continuing with KDE Activities
Hi Eric, Yes, the development had slowed down unfortunately, but expect it do liven up a bit. I'm not sure whether most immediate work will be visible to the user (apart from the things that I need to merge in this cycle that were originally developed for 5.1) but new things are comming. That will IMO benefit even people who do not use activities. Cheers, Ivan On 29 January 2015 at 11:41, Marco Martin notm...@gmail.com wrote: On Tuesday 27 January 2015 20:52:45 Eric Mesa wrote: Hello Plasma Devs, Not sure if this is the right mailing list, but I've been seeing a lot on Planet KDE from people unsure of whether Activities are used and should be kept, etc. So I wrote this specifically in hopes of making sure the devs see how it's being used by regular folks. http://www.ericsbinaryworld.com/2015/01/27/an-appeal-for-keeping-kde-activit ies/ Thank you for developing an awesome desktop environment - I enjoy it so much that it makes me sad we don't use it at work. (Well, we'd first have to start using Linux...) is true that sadly there wasn't a whole load of new development there. but in Plasma 5 a re there and here to stay -- Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel -- Cheerio, Ivan -- While you were hanging yourself on someone else's words Dying to believe in what you heard I was staring straight into the shining sun ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Proposal regarding plasmate's release
On 01/29/2015 02:52 PM, Sebastian Kügler wrote: when plasmate's new version is usable and useful, I'd like to turn this into an optional dependency, so if kdevplatform is there, we'll automatically build the plasmate IDE. +1 On 01/29/2015 02:42 PM, Marco Martin wrote: yeah, master should be in releasable state before moving. yes sorry, I meant to make the master releasable first and then to make the move. last thing, are there changes in master for kde4 not released? it may be good at least tagging it if so. yes there are. Should I just add a tag named kde4 or something like that? On 01/29/2015 02:40 PM, David Edmundson wrote: For policy reasons this requires an email to kde-core-devel Ok I can do that, thank you for letting me know. :) Which should be the new location of the plasmate repo? kde/workspace ? -- Giorgos Tsiapaliokas terietor.org ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Jenkins build is back to normal : plasma-workspace_master_qt5 #1270
See http://build.kde.org/job/plasma-workspace_master_qt5/1270/changes ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: An appeal for continuing with KDE Activities
Am Donnerstag, 29. Januar 2015, 11:41:23 schrieb Marco Martin: On Tuesday 27 January 2015 20:52:45 Eric Mesa wrote: Hello Plasma Devs, Not sure if this is the right mailing list, but I've been seeing a lot on Planet KDE from people unsure of whether Activities are used and should be kept, etc. So I wrote this specifically in hopes of making sure the devs see how it's being used by regular folks. http://www.ericsbinaryworld.com/2015/01/27/an-appeal-for-keeping-kde-a ctivit ies/ Thank you for developing an awesome desktop environment - I enjoy it so much that it makes me sad we don't use it at work. (Well, we'd first have to start using Linux...) is true that sadly there wasn't a whole load of new development there. but in Plasma 5 a re there and here to stay I am happy to read that. And I second it: I use activies myself. On the laptop with one screen and one the workstation at works in two screens. There I have a special advantage: I have some apps like mail and KDE telepathy and so on open on all apps, they are on the left screen usually. And on the right screen I have the task specific stuff. Only think I had sometimes, but its a longer time since I last had it, that after a reboot, logout, relog all windows from all activies appeared on just one activity, so it lost the assignment. And this is tedious to clean up. Thank you to all the developers working on Plasma and working on activities. I appreciate it. Ciao, -- Martin 'Helios' Steigerwald - http://www.Lichtvoll.de GPG: 03B0 0D6C 0040 0710 4AFA B82F 991B EAAC A599 84C7 ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 339427] Screen goes off when watching video using VLC
https://bugs.kde.org/show_bug.cgi?id=339427 donniezazen sud...@sudhirkhanger.com changed: What|Removed |Added Summary|Screen goes off when|Screen goes off when |watching video using VLC or |watching video using VLC |YouTube.| Status|NEEDSINFO |RESOLVED Resolution|WAITINGFORINFO |WORKSFORME --- Comment #3 from donniezazen sud...@sudhirkhanger.com --- Not able to reproduce with -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 339427] Screen goes off when watching video using VLC
https://bugs.kde.org/show_bug.cgi?id=339427 --- Comment #4 from donniezazen sud...@sudhirkhanger.com --- No longer able to reproduce it on powerdevil 5.2 -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122314: Sanitize whitespace in notification popups
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- Review request for Plasma. Bugs: 343532 https://bugs.kde.org/show_bug.cgi?id=343532 Repository: plasma-workspace Description --- While looking into https://bugs.kde.org/show_bug.cgi?id=343532 I noticed that wall sends message always starting with \n, but with our limited space, that makes no sense. With this patch the text is first trimmed from whitespace at the beginning and end, then all \ns are converted to br/ (as before), then the text replaces all remaining inner whitespace with single space and finally, if there are two new lines in succession, it turns it into just one. The bug itself, which is about messages from wall not appearing in the popup, will only partially get solved by this as the popup is limited in max height, so it displays at most 3 lines of text and with wall first two lines are already taken, that leaves just one line for the actual message. However these are persistant so they get saved in the notification history anyway. Diffs - dataengines/notifications/notificationsengine.cpp 6e566b4 Diff: https://git.reviewboard.kde.org/r/122314/diff/ Testing --- Now I'm able to see the message sent with wall. Thanks, Martin Klapetek ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Jenkins build became unstable: plasma-workspace_stable_qt5 #45
See http://build.kde.org/job/plasma-workspace_stable_qt5/45/changes ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122315: Build kwrited as kded module if kpty was built w/ utempter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122315/ --- Review request for Plasma. Repository: kwrited Description --- this was the case in all(most?) of kde4 timeline, but with the KF5 port kpty didn't expose utempter presence Diffs - CMakeLists.txt 8c29e1b Diff: https://git.reviewboard.kde.org/r/122315/diff/ Testing --- working Thanks, Hrvoje Senjan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122315: Build kwrited as kded module if kpty was built w/ utempter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122315/#review75000 --- Ship it! Ship It! - Lukáš Tinkl On Led. 29, 2015, 8:48 odp., Hrvoje Senjan wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122315/ --- (Updated Led. 29, 2015, 8:48 odp.) Review request for Plasma. Repository: kwrited Description --- this was the case in all(most?) of kde4 timeline, but with the KF5 port kpty didn't expose utempter presence Diffs - CMakeLists.txt 8c29e1b Diff: https://git.reviewboard.kde.org/r/122315/diff/ Testing --- working Thanks, Hrvoje Senjan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122315: Build kwrited as kded module if kpty was built w/ utempter
On Jan. 29, 2015, 8:53 p.m., Lukáš Tinkl wrote: Ship It! if/when r122313 gets approved, i'll push this one to master (dunno if 5.2 would be a good target?) - Hrvoje --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122315/#review75000 --- On Jan. 29, 2015, 8:48 p.m., Hrvoje Senjan wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122315/ --- (Updated Jan. 29, 2015, 8:48 p.m.) Review request for Plasma. Repository: kwrited Description --- this was the case in all(most?) of kde4 timeline, but with the KF5 port kpty didn't expose utempter presence Diffs - CMakeLists.txt 8c29e1b Diff: https://git.reviewboard.kde.org/r/122315/diff/ Testing --- working Thanks, Hrvoje Senjan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 David Edmundson k...@davidedmundson.co.uk changed: What|Removed |Added CC||alexander...@gmail.com --- Comment #63 from David Edmundson k...@davidedmundson.co.uk --- *** Bug 343542 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122318: Sort apps by name if configured
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122318/ --- Review request for Plasma. Bugs: 343538 https://bugs.kde.org/show_bug.cgi?id=343538 Repository: plasma-desktop Description --- The show apps by name option only affected the visual representation, it did not change the actual sorting. Diffs - applets/kickoff/core/applicationmodel.h e7399e8 applets/kickoff/core/applicationmodel.cpp 8d2f885 applets/kickoff/package/contents/ui/ApplicationsView.qml 9382882 Diff: https://git.reviewboard.kde.org/r/122318/diff/ Testing --- Now the items are always sorted alphabetically by whatever is written in the larger font. Thanks, Kai Uwe Broulik ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122171: Implement SystemTrayMenuItem::setMenu() correctly.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122171/#review75007 --- I noticed this got a Ship It but the original author said he had no commit access. Was this ever committed? - Luca Beltrame On Gen. 20, 2015, 3:40 p.m., Dmitry Shachnev wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122171/ --- (Updated Gen. 20, 2015, 3:40 p.m.) Review request for Plasma. Repository: frameworkintegration Description --- Implement SystemTrayMenuItem::setMenu() correctly. We need to actually set the action menu, not just store it somewhere, to make submenus in tray icons working. Diffs - src/platformtheme/kdeplatformsystemtrayicon.h 3c1bbf7 src/platformtheme/kdeplatformsystemtrayicon.cpp d3a1d4f Diff: https://git.reviewboard.kde.org/r/122171/diff/ Testing --- Tested this app: https://github.com/dglent/meteo-qt Thanks, Dmitry Shachnev ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Powerdevil] [Bug 337674] kded5 is eating CPU
https://bugs.kde.org/show_bug.cgi?id=337674 Kai Uwe Broulik k...@privat.broulik.de changed: What|Removed |Added Version Fixed In|5.2.0 |5.2.1 Resolution|--- |FIXED Status|REOPENED|RESOLVED Latest Commit|http://commits.kde.org/plas |http://commits.kde.org/powe |ma-workspace/62fbfd5b15234f |rdevil/2c2c13751e19b0a37bb8 |f38aa11ed131474991a193d17b |b41096b08f80383e61df --- Comment #62 from Kai Uwe Broulik k...@privat.broulik.de --- Git commit 2c2c13751e19b0a37bb8b41096b08f80383e61df by Kai Uwe Broulik. Committed on 29/01/2015 at 21:46. Pushed by broulik into branch 'Plasma/5.2'. Fix PowerDevil brightness calls breaking kded, Volume 3521 Query both brightness and maximum only once and then only listen to udev change events and just kick off the set job and forget about it. That way we do not have nested event loops outside the init method anymore. FIXED-IN: 5.2.1 This prevents re-entrancy issues M +23 -6daemon/actions/bundled/brightnesscontrol.cpp M +2-0daemon/actions/bundled/brightnesscontrol.h M +5-3daemon/backends/hal/powerdevilhalbackend.cpp M +1-1daemon/backends/hal/powerdevilhalbackend.h M +41 -42 daemon/backends/upower/powerdevilupowerbackend.cpp M +2-1daemon/backends/upower/powerdevilupowerbackend.h M +2-1daemon/powerdevilbackendinterface.h http://commits.kde.org/powerdevil/2c2c13751e19b0a37bb8b41096b08f80383e61df -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122314: Sanitize whitespace in notification popups
On Jan. 29, 2015, 9:31 p.m., David Edmundson wrote: dataengines/notifications/notificationsengine.cpp, line 229 https://git.reviewboard.kde.org/r/122314/diff/1/?file=345703#file345703line229 what about if you had \n\n\n I think this will still leave one. Maybe br/(\s*|br/)* oh FFS. Reviewboard removed my asterixes. https://paste.kde.org/plf9yky4o - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75004 --- On Jan. 29, 2015, 7:14 p.m., Martin Klapetek wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- (Updated Jan. 29, 2015, 7:14 p.m.) Review request for Plasma. Bugs: 343532 https://bugs.kde.org/show_bug.cgi?id=343532 Repository: plasma-workspace Description --- While looking into https://bugs.kde.org/show_bug.cgi?id=343532 I noticed that wall sends message always starting with \n, but with our limited space, that makes no sense. With this patch the text is first trimmed from whitespace at the beginning and end, then all \ns are converted to br/ (as before), then the text replaces all remaining inner whitespace with single space and finally, if there are two new lines in succession, it turns it into just one. The bug itself, which is about messages from wall not appearing in the popup, will only partially get solved by this as the popup is limited in max height, so it displays at most 3 lines of text and with wall first two lines are already taken, that leaves just one line for the actual message. However these are persistant so they get saved in the notification history anyway. Diffs - dataengines/notifications/notificationsengine.cpp 6e566b4 Diff: https://git.reviewboard.kde.org/r/122314/diff/ Testing --- Now I'm able to see the message sent with wall. Thanks, Martin Klapetek ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122314: Sanitize whitespace in notification popups
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75004 --- dataengines/notifications/notificationsengine.cpp https://git.reviewboard.kde.org/r/122314/#comment51936 what about if you had \n\n\n I think this will still leave one. Maybe br/(\s*|br/)* - David Edmundson On Jan. 29, 2015, 7:14 p.m., Martin Klapetek wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- (Updated Jan. 29, 2015, 7:14 p.m.) Review request for Plasma. Bugs: 343532 https://bugs.kde.org/show_bug.cgi?id=343532 Repository: plasma-workspace Description --- While looking into https://bugs.kde.org/show_bug.cgi?id=343532 I noticed that wall sends message always starting with \n, but with our limited space, that makes no sense. With this patch the text is first trimmed from whitespace at the beginning and end, then all \ns are converted to br/ (as before), then the text replaces all remaining inner whitespace with single space and finally, if there are two new lines in succession, it turns it into just one. The bug itself, which is about messages from wall not appearing in the popup, will only partially get solved by this as the popup is limited in max height, so it displays at most 3 lines of text and with wall first two lines are already taken, that leaves just one line for the actual message. However these are persistant so they get saved in the notification history anyway. Diffs - dataengines/notifications/notificationsengine.cpp 6e566b4 Diff: https://git.reviewboard.kde.org/r/122314/diff/ Testing --- Now I'm able to see the message sent with wall. Thanks, Martin Klapetek ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- (Updated Jan. 29, 2015, 11:15 p.m.) Status -- This change has been marked as submitted. Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[kwrited] [Bug 343550] New: duplicated kwrited instances increasing with each system boot
https://bugs.kde.org/show_bug.cgi?id=343550 Bug ID: 343550 Summary: duplicated kwrited instances increasing with each system boot Product: kwrited Version: 5.1.95 Platform: Ubuntu Packages OS: Linux Status: UNCONFIRMED Severity: normal Priority: NOR Component: general Assignee: plasma-devel@kde.org Reporter: sha...@ukr.net Each time i reboot my ubuntu vivid alpha with plasma 5.2.0 packages one additional instance of kwrited is added by a session restore. i remember seeing a code to avoid this in some other apps, but it seems that this code should be added all over the places where session restore is used Reproducible: Always Steps to Reproduce: ps aux | grep kwrited a sudo reboot ps aux | grep kwrited b see a and b in attachment -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122314: Sanitize whitespace in notification popups
On Jan. 29, 2015, 10:31 p.m., David Edmundson wrote: dataengines/notifications/notificationsengine.cpp, line 229 https://git.reviewboard.kde.org/r/122314/diff/1/?file=345703#file345703line229 what about if you had \n\n\n I think this will still leave one. Maybe br/(\s*|br/)* David Edmundson wrote: oh FFS. Reviewboard removed my asterixes. https://paste.kde.org/plf9yky4o Ah right, yeah. Actually the asterisk just outside the parenthesis will do. (though that will replace also all single br/, not sure how expensive this can be; additionally could be like ```br/\s*br/(\s|br/)*``` ...) - Martin --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75004 --- On Jan. 29, 2015, 8:14 p.m., Martin Klapetek wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- (Updated Jan. 29, 2015, 8:14 p.m.) Review request for Plasma. Bugs: 343532 https://bugs.kde.org/show_bug.cgi?id=343532 Repository: plasma-workspace Description --- While looking into https://bugs.kde.org/show_bug.cgi?id=343532 I noticed that wall sends message always starting with \n, but with our limited space, that makes no sense. With this patch the text is first trimmed from whitespace at the beginning and end, then all \ns are converted to br/ (as before), then the text replaces all remaining inner whitespace with single space and finally, if there are two new lines in succession, it turns it into just one. The bug itself, which is about messages from wall not appearing in the popup, will only partially get solved by this as the popup is limited in max height, so it displays at most 3 lines of text and with wall first two lines are already taken, that leaves just one line for the actual message. However these are persistant so they get saved in the notification history anyway. Diffs - dataengines/notifications/notificationsengine.cpp 6e566b4 Diff: https://git.reviewboard.kde.org/r/122314/diff/ Testing --- Now I'm able to see the message sent with wall. Thanks, Martin Klapetek ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/#review75010 --- Ship it! Ship It! - Lukáš Tinkl On Led. 29, 2015, 11:56 odp., Rajeesh K Nambiar wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- (Updated Led. 29, 2015, 11:56 odp.) Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122171: Implement SystemTrayMenuItem::setMenu() correctly.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122171/ --- (Updated Jan. 29, 2015, 11:08 p.m.) Status -- This change has been marked as submitted. Review request for Plasma. Repository: frameworkintegration Description --- Implement SystemTrayMenuItem::setMenu() correctly. We need to actually set the action menu, not just store it somewhere, to make submenus in tray icons working. Diffs - src/platformtheme/kdeplatformsystemtrayicon.h 3c1bbf7 src/platformtheme/kdeplatformsystemtrayicon.cpp d3a1d4f Diff: https://git.reviewboard.kde.org/r/122171/diff/ Testing --- Tested this app: https://github.com/dglent/meteo-qt Thanks, Dmitry Shachnev ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/#review75012 --- src/kded/kdedactions.cpp https://git.reviewboard.kde.org/r/122319/#comment51940 Use qWarning rather than qDebug, I'd say it's a warning after all... src/kded/kdedactions.cpp https://git.reviewboard.kde.org/r/122319/#comment51941 If this is not needed anymore, just remove it altogether - Martin Klapetek On Jan. 29, 2015, 11:56 p.m., Rajeesh K Nambiar wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- (Updated Jan. 29, 2015, 11:56 p.m.) Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
On Led. 30, 2015, 12:07 dop., Lukáš Tinkl wrote: Ship It! Rajeesh K Nambiar wrote: That was quick! :-) I don't have commit access - could you commit please? Sure; why not ask for an account then? :) - Lukáš --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/#review75010 --- On Led. 29, 2015, 11:56 odp., Rajeesh K Nambiar wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- (Updated Led. 29, 2015, 11:56 odp.) Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122319: Set global shortcuts for touchpad enable/disable/toggle
On Jan. 29, 2015, 11:07 p.m., Lukáš Tinkl wrote: Ship It! That was quick! :-) I don't have commit access - could you commit please? - Rajeesh --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/#review75010 --- On Jan. 29, 2015, 10:56 p.m., Rajeesh K Nambiar wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122319/ --- (Updated Jan. 29, 2015, 10:56 p.m.) Review request for Plasma, Alexander Mezin and Jonathan Riddell. Bugs: 342629 http://bugs.kde.org/show_bug.cgi?id=342629 Repository: kcm-touchpad Description --- Set global shortcuts for touchpad enable/disable/toggle. Review and comments welcome. Bug 342629 Diffs - src/kded/kdedactions.cpp ca32834 Diff: https://git.reviewboard.kde.org/r/122319/diff/ Testing --- Build Fedora RPMS against Plasma 5.2, default global shortcuts for touchpad are set. Thanks, Rajeesh K Nambiar ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[Breeze] [Bug 343555] New: Breeze support for gtk2 and gtk3
https://bugs.kde.org/show_bug.cgi?id=343555 Bug ID: 343555 Summary: Breeze support for gtk2 and gtk3 Product: Breeze Version: unspecified Platform: Archlinux Packages OS: Linux Status: UNCONFIRMED Severity: wishlist Priority: NOR Component: general Assignee: plasma-devel@kde.org Reporter: llkiwi2...@gmail.com It would be helpful for breeze to support gtk2 and gtk3 like oxygen, so that having an uniform look and feel for both qt and gtk applications while using breeze is possible. Reproducible: Always -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[kwrited] [Bug 343550] duplicated kwrited instances increasing with each system boot
https://bugs.kde.org/show_bug.cgi?id=343550 --- Comment #1 from Nick Shaforostoff sha...@ukr.net --- Created attachment 90802 -- https://bugs.kde.org/attachment.cgi?id=90802action=edit ps aux | grep kwrited a -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
[kwrited] [Bug 343550] duplicated kwrited instances increasing with each system boot
https://bugs.kde.org/show_bug.cgi?id=343550 --- Comment #2 from Nick Shaforostoff sha...@ukr.net --- Created attachment 90803 -- https://bugs.kde.org/attachment.cgi?id=90803action=edit ps aux | grep kwrited b -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel