Extra patch for KConfig 5.7 release

2015-02-13 Thread Matthew Dawson
Hi all, If it is not too late to sneak an extra patch into KConfig's 5.7 release, could commit 9eee15917e01a89d937d1cba2eebbe9d65daeb72 ( http://commits.kde.org/kconfig/9eee15917e01a89d937d1cba2eebbe9d65daeb72 ) be added in? This change reverts a SIC, which will break updating of configuration

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Ben Cooksley
On Sat, Feb 14, 2015 at 8:17 AM, Marko Käning wrote: > Hi, Hi Marko, > > On 13 Feb 2015, at 10:15 , Kevin Funk wrote: >> Indeed, the kde/* rule in 'logical-module-structure' is wrong already: > > yep, that’s what I wanted to express. =) > > Ben has fixed that in the meantime [1] and then chang

Re: Review Request 122555: knotifications: Add optional dependency on Qt5TextToSpeech for speech notifications.

2015-02-13 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122555/ --- (Updated Feb. 13, 2015, 4:01 p.m.) Review request for KDE Frameworks and

Re: Review Request 122551: New feature: Open all recent files

2015-02-13 Thread Gregor Mi
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122551/#review76011 --- This only covers your point 3): You added a menu item "Open a

Re: Review Request 122551: New feature: Open all recent files

2015-02-13 Thread Thomas Murach
> On Feb. 13, 2015, 12:06 vorm., Kevin Funk wrote: > > You're aware that Kate (and any other decent editor) has session management > > which probably solves your issue? It saves the session when you close it, > > and re-opens all the files that had been opened. Sure, I use sessions quite frequ

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Frederik Gladhorn
> On Feb. 13, 2015, 11:49 a.m., Aleix Pol Gonzalez wrote: > > +1 > > > > What's the status of QtSpeech? > > Jeremy Whiting wrote: > I'll blog about it shortly. I think QtSpeech is Qt 5.6 material realistically. It's usable on most platforms but needs cleanup and API review and a couple of

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Frederik Gladhorn
> On Feb. 13, 2015, 12:17 p.m., Martin Klapetek wrote: > > src/knotifyconfigactionswidgetbase.ui, line 141 > > > > > > Is Jovie still used in QtSpeech? If not, it should be removed from here I tend to agree to T

Re: Review Request 122555: knotifications: Add optional dependency on Qt5TextToSpeech for speech notifications.

2015-02-13 Thread Frederik Gladhorn
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122555/#review76006 --- src/notifybyktts.h

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122554/ --- (Updated Feb. 13, 2015, 12:42 p.m.) Review request for KDE Frameworks and

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Jeremy Whiting
> On Feb. 13, 2015, 4:49 a.m., Aleix Pol Gonzalez wrote: > > +1 > > > > What's the status of QtSpeech? I'll blog about it shortly. - Jeremy --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org

Re: Review Request 122555: knotifications: Add optional dependency on Qt5TextToSpeech for speech notifications.

2015-02-13 Thread Jeremy Whiting
> On Feb. 13, 2015, 5:11 a.m., Martin Klapetek wrote: > > src/notifybyspeech.cpp, line 65 > > > > > > Is there any way to know when the say() has finished? Because the > > finished() below will delete the notific

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Marko Käning
Hi, On 13 Feb 2015, at 10:15 , Kevin Funk wrote: > Indeed, the kde/* rule in 'logical-module-structure' is wrong already: yep, that’s what I wanted to express. =) Ben has fixed that in the meantime [1] and then changed it later again [2]. So, now one would have to introduce for all the below

Review Request 122562: Add method for converting from QVariant to base64

2015-02-13 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122562/ --- Review request for KDE Frameworks and Plasma. Repository: kdeclarative

Re: KPeople part of KDE Frameworks

2015-02-13 Thread Aleix Pol
On Fri, Feb 13, 2015 at 2:25 PM, Aleix Pol wrote: > Hi, > I would like to submit KPeople for review process so it can join the > KF5 family in the next 5.8 release. > > KPeople is a tier 3 framework (because of KService, which will be > dropped eventually, without breaking ABI) and it offers cross

Jenkins build is back to normal : kjs_stable_qt5 #10

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Jenkins build is back to normal : kjs_master_qt5 #72

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Jenkins build is back to normal : frameworkintegration_stable_qt5 #31

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: kjs_master_qt5 #71

2015-02-13 Thread KDE CI System
See Changes: [aacid] add missing man page docbook -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 1 (PACKAGER LINBUILDER) in workspace

Build failed in Jenkins: kjs_stable_qt5 #9

2015-02-13 Thread KDE CI System
See Changes: [aacid] add missing man page docbook -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 1 (PACKAGER LINBUILDER) in workspace

Jenkins build is back to stable : plasma-framework_master_qt5 » All,LINBUILDER #999

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 117985: frameworks/kjs: add missing man page docbook

2015-02-13 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117985/ --- (Updated Feb. 13, 2015, 1:29 p.m.) Status -- This change has been ma

KPeople part of KDE Frameworks

2015-02-13 Thread Aleix Pol
Hi, I would like to submit KPeople for review process so it can join the KF5 family in the next 5.8 release. KPeople is a tier 3 framework (because of KService, which will be dropped eventually, without breaking ABI) and it offers cross-source contact aggregation. More information about it can be

Jenkins build became unstable: kcodecs_master_qt5 #82

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: frameworkintegration_stable_qt5 #30

2015-02-13 Thread KDE CI System
See Changes: [Kevin Funk] Use Q_DECL_OVERRIDE where possible -- [...truncated 87 lines...] -- Check for working C compiler: /home/jenkins/bin/cc -- Check for working C compiler: /home/je

Re: Review Request 122542: All frameworks: Add Q_DECL_OVERRIDE where needed

2015-02-13 Thread Kevin Funk
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122542/ --- (Updated Feb. 13, 2015, 12:40 p.m.) Status -- This change has been m

Jenkins build is still unstable: plasma-framework_master_qt5 » All,LINBUILDER #998

2015-02-13 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122554/#review75987 --- As I said in the other review, I believe the user facing strin

Re: Review Request 122555: knotifications: Add optional dependency on Qt5TextToSpeech for speech notifications.

2015-02-13 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122555/#review75986 --- Looks good, thanks for that! I'm wondering if using TTS in the

Re: Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.

2015-02-13 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122554/#review75983 --- +1 What's the status of QtSpeech? - Aleix Pol Gonzalez On

Re: New framework: KXmlRpcClient

2015-02-13 Thread Martin Klapetek
On Fri, Feb 13, 2015 at 10:34 AM, Harald Sitter wrote: > > > > Can someone tell the distro packagers list they'll have a duplciation > issue > > when using frameworks 5.8 + Plasma 5.2? > > if the translation catalog gets renamed there is no conflict. > currently it is libkxmlrpcclient5.pot, so si

Re: New framework: KXmlRpcClient

2015-02-13 Thread Harald Sitter
On Thu, Feb 12, 2015 at 8:25 PM, Albert Astals Cid wrote: > El Dijous, 12 de febrer de 2015, a les 12:31:14, Daniel Vrátil va escriure: >> On Friday, January 30, 2015 09:12:01 AM David Faure wrote: >> > On Wednesday 28 January 2015 17:12:04 Daniel Vrátil wrote: >> > > I guess I should update kde-b

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Ben Cooksley
On Fri, Feb 13, 2015 at 10:15 PM, Kevin Funk wrote: > On Friday 13 February 2015 21:56:42 Ben Cooksley wrote: >> On Fri, Feb 13, 2015 at 9:52 PM, Kevin Funk wrote: >> > On Friday 13 February 2015 08:49:10 Marko Käning wrote: >> >> Hi Albert, >> >> >> >> I just realised that kde-baseapps fails on

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Kevin Funk
On Friday 13 February 2015 21:56:42 Ben Cooksley wrote: > On Fri, Feb 13, 2015 at 9:52 PM, Kevin Funk wrote: > > On Friday 13 February 2015 08:49:10 Marko Käning wrote: > >> Hi Albert, > >> > >> I just realised that kde-baseapps fails on OSX/CI when being build for > >> branch group "stable-kf5-q

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Ben Cooksley
On Fri, Feb 13, 2015 at 9:52 PM, Kevin Funk wrote: > On Friday 13 February 2015 08:49:10 Marko Käning wrote: >> Hi Albert, >> >> I just realised that kde-baseapps fails on OSX/CI when being build for >> branch group "stable-kf5-qt5": >> >> -- >> Group: s

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Kevin Funk
On Friday 13 February 2015 08:49:10 Marko Käning wrote: > Hi Albert, > > I just realised that kde-baseapps fails on OSX/CI when being build for > branch group "stable-kf5-qt5": > > -- > Group: stable-kf5-qt5 > Project : kde/applications/kde-baseapps >

Re: OSX/CI: kde-baseapps fails to build for branch-group "stable-kf5-qt5"

2015-02-13 Thread Marko Käning
OK, I understood only now, that kde-baseapps [1] and likely also the other two projects are KDE4-only. There is no EMPTY entry for “stable-kd5-qt5” branch group for these three projects in our logical dependencies, which should be the reason for this. Should I add those? (I suspect also that