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 t

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 possib

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 > http://build.kde.org/job/kde-baseapps_mast

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

2015-02-13 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122563/ --- (Updated Feb. 13, 2015, 8:47 p.m.) Status -- This change has been ma

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 13

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, 12:19

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

2015-02-13 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122563/ --- Review request for KDE Software on Mac OS X and kdelibs. Repository: kdel

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

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 copy&paste: the copies of KDChart in Calligra & KMyMoney > > > are older (2.4

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: 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 ma

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 > http://build.kde.org/job/kde-baseapps_mast

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 a.

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 > http://build.kde.org/job/kde-baseapps_mast

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 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 sho

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