Jenkins-kde-ci: plasma-workspace Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 77 - Still Unstable!

2017-02-19 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-workspace%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/77/ Project: PLATFORM=Linux,compiler=gcc Date of build: Mon, 20 Feb 2017 06:45:00 + Build duration: 8 min 19 sec CHANGE SET Revision 3d960a2f7a5e8182bd

Jenkins-kde-ci: plasma-desktop Plasma-5.9 stable-kf5-qt5 » Linux,gcc - Build # 76 - Still Unstable!

2017-02-19 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-desktop%20Plasma-5.9%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/76/ Project: PLATFORM=Linux,compiler=gcc Date of build: Mon, 20 Feb 2017 06:44:12 + Build duration: 9 min 12 sec CHANGE SET Revision c40fe1f37ba591eed32f

Jenkins-kde-ci: plasma-desktop master kf5-qt5 » Linux,gcc - Build # 625 - Failure!

2017-02-19 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-desktop%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/625/ Project: PLATFORM=Linux,compiler=gcc Date of build: Mon, 20 Feb 2017 05:01:36 + Build duration: 3 min 0 sec CHANGE SET Revision 19441cbf9d5f9e763e3063b4fdf3a47d

[Differential] [Commented On] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread Anthony Fieroni
anthonyfieroni added a comment. OK, upgrading just override existing library, no? So lets make 2 test: 1. Change in taskmanagerplugin that isn't abi -> force replace the lib when plasmashell works -> nothing happened 2. Make an abi change -> force replace it when plasmashell works -> cr

Jenkins-kde-ci: khotkeys master kf5-qt5 » Linux,gcc - Build # 33 - Failure!

2017-02-19 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/khotkeys%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/33/ Project: PLATFORM=Linux,compiler=gcc Date of build: Mon, 20 Feb 2017 05:00:51 + Build duration: 1 min 58 sec CHANGE SET Revision 9dee35bf3166c44484a9377069e9014b5c0155

[Differential] [Request, 8 lines] D4671: Touchpad KCM: simplify few UI strings

2017-02-19 Thread Luigi Toscano
ltoscano created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D4671 AFFECTED FILES kcms/touchpad/src/kcm/libinput/touchpadc

[Differential] [Commented On] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread David Edmundson
davidedmundson added a comment. > David Ed: you're right, but think of the case where you open a new plugin (in a long running process) which itself links to a freshly upgraded shared lib - I'm not sure what happens but it's one of these two cases: That's a good question; though it seems

[Differential] [Reopened] D4491: Let make taskmanager tooltip readable again

2017-02-19 Thread Mark Gaiser
markg reopened this revision. markg added a comment. This revision is now accepted and ready to land. So now i'm on the correct revision it seems. I applied the diff locally to see how this change looks. It looks OK (can't test it on a "retina" display), but still quite inconsistent with

[Differential] [Commented On] D3738: [Task Manager] Tooltips redesign

2017-02-19 Thread Mark Gaiser
markg added a comment. In https://phabricator.kde.org/D3738#87617, @subdiff wrote: > In https://phabricator.kde.org/D3738#87601, @markg wrote: > > > ... > > > There has already been a commit to master tackling this issue: https://phabricator.kde.org/D4491 That's great, t

[Differential] [Closed] D3738: [Task Manager] Tooltips redesign

2017-02-19 Thread Roman Gilg
subdiff closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3738 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: subdiff, #vdg, hein, #plasma Cc: markg, broulik, anthonyfieroni, hein, colomar, plasma-devel, p

[Differential] [Commented On] D3738: [Task Manager] Tooltips redesign

2017-02-19 Thread Roman Gilg
subdiff added a comment. In https://phabricator.kde.org/D3738#87601, @markg wrote: > ... There has already been a commit to master tackling this issue: https://phabricator.kde.org/D4491 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3738 EMAIL P

[Differential] [Closed] D4538: [KTextEditor] consistent conversion from/to cursor to/from coordinates

2017-02-19 Thread John Salatas
This revision was automatically updated to reflect the committed changes. Closed by commit R39:1858d6fafb58: Consistent conversion from/to cursor to/from coordinates (authored by jsalatas). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4538?vs=11494&id=115

[Maniphest] [Commented On] T5412: Improve Plasma PA usability

2017-02-19 Thread David Rosca
drosca added a comment. > No switches to quickly change the default output directly from the applet. This needs to go in, I forgot about it: https://phabricator.kde.org/D2314 > Middle click for muting is not intuitive, left click on indicator icon difficult because of size. You

[Differential] [Commented On] D1771: [Workspace Options] Add option to disable OSD

2017-02-19 Thread Martin Gräßlin
graesslin added a comment. This one is pushed, isn't it? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1771 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma:_design, graesslin, #plasma Cc: andreaska, colo

[Differential] [Reopened] D3738: [Task Manager] Tooltips redesign

2017-02-19 Thread Mark Gaiser
markg reopened this revision. markg added a comment. This revision is now accepted and ready to land. Hmm, i don't know if this is the appropriate way in a phabricator workflow. But this does reach exactly those involved in this change which is what i intent. On to the point. When this

[Maniphest] [Commented On] T5412: Improve Plasma PA usability

2017-02-19 Thread Kai Uwe Broulik
broulik added a comment. > No button to open the related KCM. Could be fit next to the tab bar, I suppose. > Device icons are ambiguous or wrong: That's unfortunately mostly due to a limitation of PulseAudio, it doesn't even distinguish between HDMI and DisplayPort… > No sw

[Maniphest] [Commented On] T5412: Improve Plasma PA usability

2017-02-19 Thread Kai Uwe Broulik
broulik added a comment. > Device icons are ambiguous or wrong: > Default to not change streams on newly connected inputs/outputs is maybe not the right default. Definitely. We have this option now but for some reason we didn't make it the default. When I plug in my TV I want

[Maniphest] [Created] T5412: Improve Plasma PA usability

2017-02-19 Thread Roman Gilg
subdiff created this task. subdiff added projects: Plasma, VDG. TASK DESCRIPTION There have been huge improvements to Plasma's audio handling in the past thanks to @drosca. But there is still room for more. Currently I see the following issues, which in some cases might be easy to fix, in othe

Looking to pick up some tasks

2017-02-19 Thread Yuen Hoe Lim
Hey all! I'd like to get started with contributing code and doing some tasks again so thought I'd write in. Not sure if anyone remembers me :) I did a bit of work on Plasma before quite a few years ago and was also a summer of code student - all the way back during the very first Plasma Mobile on

[Differential] [Commented On] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread Anthony Fieroni
anthonyfieroni added a comment. > @sitter: **BUT** the user can still add (potentially new) applets to the shell which in turn is then broken because of the module Plasma refuses to load because of version-locking. I think that's not a problem, old plasma can load only old plugins, user

[Differential] [Accepted] D4538: [KTextEditor] consistent conversion from/to cursor to/from coordinates

2017-02-19 Thread Christoph Cullmann
cullmann accepted this revision. cullmann added a comment. This revision is now accepted and ready to land. Thanks for extending the test case + trying the KDevelop navigation! REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D4538 EMAIL PREFERENCES https://phabri

Playing with libseccomp

2017-02-19 Thread Martin Gräßlin
Hi all, this weekend I spent some time on trying to get kscreenlocker_greet into a sandbox through libseccomp. My aim was to make it impossible for a lnf package to intercept the password and e.g. write it to a file. In this mail I want to outline my findings. First of all, what does libsecc

[Differential] [Updated] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread Harald Sitter
sitter added a comment. In https://phabricator.kde.org/D4667#87551, @dfaure wrote: > I was always told "plugins should not get versioned at the .so level". Not sure why exactly. Are you sure this wasn't simply about plugins not being in public library directories? I know some peo

[kio-extras] [Bug 376577] Search crashes when started from root folder

2017-02-19 Thread Elvis Angelaccio
https://bugs.kde.org/show_bug.cgi?id=376577 Elvis Angelaccio changed: What|Removed |Added Product|dolphin |kio-extras Component|general

[kio-extras] [Bug 376577] filenamesearch crashes when started from root folder

2017-02-19 Thread Elvis Angelaccio
https://bugs.kde.org/show_bug.cgi?id=376577 Elvis Angelaccio changed: What|Removed |Added Summary|Search crashes when started |filenamesearch crashes when

[Differential] [Updated] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread David Faure
dfaure added a comment. I was always told "plugins should not get versioned at the .so level". Not sure why exactly. The more usual solution was version numbers in .desktop files, to query for things with the right version. I guess this can be transposed to json since Qt reads the json data

[Differential] [Updated] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread Anthony Fieroni
anthonyfieroni added a reviewer: dfaure. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4667 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: anthonyfieroni, #plasma, sitter, davidedmundson, dfaure Cc: plasma-devel, progwolff,

[Differential] [Commented On] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread Anthony Fieroni
anthonyfieroni added a comment. Let try to better clarification, we breaking abi like this (https://cgit.kde.org/plasma-desktop.git/commit/?h=Plasma/5.9&id=dc17f78ebcc76600040a8d2ef8c57b9a41a8d06e) running update from 5.9.1 to 5.9.2 cause a crash in plasmashell, i don't save backtrace but cr

[Differential] [Updated] D4667: [applet] Let specify a version for applets private plugins

2017-02-19 Thread David Edmundson
davidedmundson added a comment. That's not how the kernel works. If it opens a file and then the file gets deleted/replaced the original file will only actually be deleted when any open references are dropped. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4