Re: Proposal: move KWalletManager inside KWallet framework

2015-06-13 Thread Valentin Rusu
* Valentin Rusu k...@rusu.info [2015-06-13 18:29:46 +0200]: * Stefan Bruens stefan.bru...@rwth-aachen.de [2015-06-13 18:16:21 +0200]: On Tuesday 02 June 2015 23:15:46 Stefan Bruens wrote: On Friday 29 May 2015 22:31:00 Valentin Rusu wrote: Ok, I should also test this. Do you have a

Re: Review Request 123692: autotests: Use QTEST_GUILESS_MAIN

2015-06-13 Thread Heiko Becker
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123692/#review81439 --- Ping? - Heiko Becker On Mai 8, 2015, 7:15 nachm., Heiko

Re: Proposal: move KWalletManager inside KWallet framework

2015-06-13 Thread Stefan Bruens
On Tuesday 02 June 2015 23:15:46 Stefan Bruens wrote: On Friday 29 May 2015 22:31:00 Valentin Rusu wrote: Ok, I should also test this. Do you have a KDE commiter account? Can you push a branch with your patched code, in the KWallet framework, please? I have pushed a

KWalletManager frameworks branch merged to master

2015-06-13 Thread Valentin Rusu
Hello, Please be advised that kdeutils/kwalletmanager is now KF5 based as I just merged the frameworks branch to the master branch. -- Valentin Rusu IRC: valir ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org

Re: Review Request 123075: do not require X11 on Mac OS X

2015-06-13 Thread Marko Käning
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote: as in other similar requests: -2 from my side Martin Gräßlin wrote: To extend: I think the way is wrong. If it now builds on MacOS the required is wrong. It should be an optional find_package properly ifdefed. Christoph Cullmann

Re: Review Request 123692: autotests: Use QTEST_GUILESS_MAIN

2015-06-13 Thread Marko Käning
On June 13, 2015, 10:26 a.m., Heiko Becker wrote: Ping? Hi Heiko, I've added this to our KDE's Phabricator task [T398](https://phabricator.kde.org/T398). ATM I can't test on my own OSX/CI system, as that is in limbo since we changed to Scarlett's new KDE/CI system. But I'll come back to

Re: Review Request 123692: autotests: Use QTEST_GUILESS_MAIN

2015-06-13 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123692/#review81446 --- Ship it! This is making it run without a gui on linux fine.

Jenkins-kde-ci: ktextwidgets master stable-kf5-qt5 » Linux,gcc - Build # 11 - Fixed!

2015-06-13 Thread no-reply
We thank you for fixing your build!___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 124089: Connected timeout before timer start.

2015-06-13 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124089/#review81450 --- startTimer is called many times so doing the connect every

Jenkins-kde-ci: knewstuff master stable-kf5-qt5 » OSX,clang - Build # 30 - Failure!

2015-06-13 Thread no-reply
Check console output at https://build.kde.org/job/knewstuff%20master%20stable-kf5-qt5/PLATFORM=OSX,compiler=clang/30/ to view the results. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org

Re: Review Request 123692: autotests: Use QTEST_GUILESS_MAIN

2015-06-13 Thread Heiko Becker
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123692/ --- (Updated June 13, 2015, 7:56 p.m.) Status -- This change has been

Re: Review Request 123692: autotests: Use QTEST_GUILESS_MAIN

2015-06-13 Thread Heiko Becker
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123692/#review81451 --- I've pushed this after Jeremy's comment. As someone who has

Jenkins-kde-ci: knewstuff master stable-kf5-qt5 » OSX,clang - Build # 31 - Fixed!

2015-06-13 Thread no-reply
We thank you for fixing your build!___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 123075: do not require X11 on Mac OS X

2015-06-13 Thread Martin Gräßlin
On March 20, 2015, 8:07 a.m., Martin Gräßlin wrote: as in other similar requests: -2 from my side Martin Gräßlin wrote: To extend: I think the way is wrong. If it now builds on MacOS the required is wrong. It should be an optional find_package properly ifdefed. Christoph Cullmann