Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122556/#review75971 --- AFAIK our CI system doesn't support Qt 5.4 yet. I think we

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Kevin Funk
On Feb. 13, 2015, 8:41 a.m., Martin Gräßlin wrote: AFAIK our CI system doesn't support Qt 5.4 yet. I think we should first ensure the CI system is prepared for building with the new dependency. Albert Astals Cid wrote: It does

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Albert Astals Cid
On feb. 13, 2015, 8:41 a.m., Martin Gräßlin wrote: AFAIK our CI system doesn't support Qt 5.4 yet. I think we should first ensure the CI system is prepared for building with the new dependency. It does http://build.kde.org/job/kde-baseapps_master_qt5/238/consoleFull qt5 - Branch

Re: Review Request 122552: warnings--

2015-02-13 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122552/#review75975 --- Ship it! Ship It! - Marco Martin On Feb. 13, 2015, 1:49

Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Arjun AK
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122556/ --- Review request for KDE Base Apps. Repository: kde-baseapps Description

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Aleix Pol Gonzalez
On Feb. 13, 2015, 9:41 a.m., Martin Gräßlin wrote: AFAIK our CI system doesn't support Qt 5.4 yet. I think we should first ensure the CI system is prepared for building with the new dependency. Albert Astals Cid wrote: It does

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

Re: KPeople part of KDE Frameworks

2015-02-13 Thread Aleix Pol
On Fri, Feb 13, 2015 at 2:25 PM, Aleix Pol aleix...@kde.org 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

Re: Review Request 122552: warnings--

2015-02-13 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122552/ --- (Updated Feb. 13, 2015, 1:17 p.m.) Status -- This change has been

Re: KDiagram libs (KChart, KGantt) in KDE Review

2015-02-13 Thread Friedrich W. H. Kossebau
Am Mittwoch, 11. Februar 2015, 16:30:39 schrieb Andreas Hartmetz: On Wednesday 11 February 2015 14:59:50 Adriaan de Groot wrote: On Monday 09 February 2015 01:50:03 Friedrich W. H. Kossebau wrote: Yes, nearly copypaste: the copies of KDChart in Calligra KMyMoney are older (2.4.1, based

Re: Review Request 122552: warnings--

2015-02-13 Thread Marco Martin
I did some tests. the correct url is http://localhost:3000/bodega/v1/json/route even tough no routes seems to match I can't figure out why, the only thing seems to be the express module to be broken in some way. probably good to make some tests with simple exampl-ish express apps On Fri, Feb

Re: Review Request 122563: [OS X] emulate native window settings restore (size and position)

2015-02-13 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122563/#review76003 --- Ship it! Ship It! - Jeremy Whiting On Feb. 13, 2015,

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Marko Käning
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122556/#review76012 --- I'd vote for keeping it compatible to Qt 5.3, if that's

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Ben Cooksley
On Feb. 13, 2015, 8:41 a.m., Martin Gräßlin wrote: AFAIK our CI system doesn't support Qt 5.4 yet. I think we should first ensure the CI system is prepared for building with the new dependency. Albert Astals Cid wrote: It does

Re: Review Request 122556: Bump Qt version to 5.4

2015-02-13 Thread Inge Wallin
On Feb. 13, 2015, 11:39 p.m., Marko Käning wrote: I'd vote for keeping it compatible to Qt 5.3, if that's possible. +1. Depending on 5.4 in applications already is bad enough even though it can sometimes be necessary but I am against it for our base stuff. Especially in this case where