Re: Review Request 127245: Fix Kdelibs4 config migration for Windows

2016-03-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127245/#review93083 --- Ship it! Thanks, looks more correct indeed. - David Faure

Re: Review Request 127261: Fix dead lock when program use kauth exits.

2016-03-02 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127261/#review93082 --- Ship it! nice work! I had run into it in the past and didn

Re: Review Request 127261: Fix dead lock when program use kauth exits.

2016-03-02 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127261/ --- (Updated March 2, 2016, 8:58 p.m.) Review request for KDE Frameworks and

Review Request 127261: Fix dead lock when program use kauth exits.

2016-03-02 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127261/ --- Review request for KDE Frameworks and David Edmundson. Repository: kauth

Re: Review Request 127205: Add stubs to allow compilation on Android.

2016-03-02 Thread Andreas Cord-Landwehr
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127205/ --- (Updated März 2, 2016, 8:18 nachm.) Review request for KDE Frameworks, Al

Re: Review Request 127154: GUI option to configure global MarkPartial for KIO slaves

2016-03-02 Thread Martin Kostolný
> On Feb. 27, 2016, 11:11 a.m., David Faure wrote: > > It seems weird to have a setting for FTP on one side, and a setting for > > "SMB and SFTP and any slave where this is implemented in the future". > > I understand that it was the path of least code changes, but I fear this > > creates an in

Re: Review Request 127154: GUI option to configure global MarkPartial for KIO slaves

2016-03-02 Thread David Faure
> On March 2, 2016, 2:26 p.m., Marco Martin wrote: > > I find a bit baffling that an option is needed for this... You are not exactly suggesting a better solution, though. Options are not needed when we can make everything work magically out of the box. However FTP servers are not exactly und

Re: Review Request 127154: GUI option to configure global MarkPartial for KIO slaves

2016-03-02 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127154/#review93075 --- I find a bit baffling that an option is needed for this... -

Re: Review Request 127223: Don't emit destroyedChanged(true) twice

2016-03-02 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127223/#review93074 --- Ship it! Ship It! - Marco Martin On Feb. 29, 2016, 4:19

Re: Review Request 127205: Add stubs to allow compilation on Android.

2016-03-02 Thread Alex Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127205/#review93073 --- Looks good to me. I'd prefer inline noop functions to macros

Re: Review Request 127251: [KUnitConversion] Fix downloading currency exchange rates

2016-03-02 Thread Kai Uwe Broulik
> On März 2, 2016, 9 vorm., Ben Cooksley wrote: > > As a warning (because you'll be using QNAM here) please ensure you handle > > redirects properly and completely. Any software which fails to do so is > > considered unsupported even before it is released in regards to accessing > > KDE.org in

Re: Review Request 127215: simplify code, reduce pointer dereferences

2016-03-02 Thread Nick Shaforostoff
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127215/ --- (Updated March 2, 2016, 12:03 p.m.) Status -- This change has been m

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 289 - Fixed!

2016-03-02 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/289/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 02 Mar 2016 11:15:19 + Build duration: 3 min 22 sec CHANGE SET Revision c

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 289 - Fixed!

2016-03-02 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/289/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 02 Mar 2016 11:15:19 + Build duration: 3 min 22 sec CHANGE SET Revision c

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 296 - Fixed!

2016-03-02 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/296/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 02 Mar 2016 11:15:19 + Build duration: 2 min 42 sec CHANGE SET

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 296 - Fixed!

2016-03-02 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/296/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 02 Mar 2016 11:15:19 + Build duration: 2 min 42 sec CHANGE SET

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,All,gcc - Build # 296 - Still Failing!

2016-03-02 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/296/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 02 Mar 2016 11:15:19 + Build duration: 1 min 10 sec CHANGE SET Rev

Re: Review Request 127245: Fix Kdelibs4 config migration for Windows

2016-03-02 Thread Andre Heinecke
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127245/ --- (Updated March 2, 2016, 11:04 a.m.) Review request for KDE Frameworks.

Re: Review Request 127245: Fix Kdelibs4 config migration for Windows

2016-03-02 Thread Andre Heinecke
> On March 2, 2016, 8:19 a.m., David Faure wrote: > > I couldn't find "AppData" or "Roaming" in kdelibs4 though. Did this come > > from windows API calls, or from kdewin maybe? Just wanted to check it was > > the exact same logic as in kdelibs4... In kdecore/kernel/kstandarddirs.cpp line 1733

Re: Review Request 127251: [KUnitConversion] Fix downloading currency exchange rates

2016-03-02 Thread Ben Cooksley
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127251/#review93058 --- As a warning (because you'll be using QNAM here) please ensur

Re: KWidgetsAddons License (included breeze icons)

2016-03-02 Thread David Faure
On Tuesday 01 March 2016 10:48:16 Andre Heinecke wrote: > Hi, > > I want to implement a similiar "Show Password" functionality as > KPasswordDialog from KWidgetsAddons has in GnuPG's pinentry. > > I'd like to use the same Icons for this, but Pinentry is GPLv2 or Later while > breeze is LGPLv3.

Re: Review Request 127236: Leverage QDir to know what's in a KIconThemeDir

2016-03-02 Thread David Faure
> On March 1, 2016, 4:37 p.m., Aleix Pol Gonzalez wrote: > > Eh... I just realized it's not 100% correct. We have a test > > (testUnknownIconNotCached) that fails, unsure how I didn't see it yesterday. > > > > The problem with this one is that we are not reacting when icons are > > introduced

Re: Review Request 127215: simplify code, reduce pointer dereferences

2016-03-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127215/#review93056 --- Ship it! Well, the Qt bug isn't invalid, there's a bug if

Re: Review Request 127245: Fix Kdelibs4 config migration for Windows

2016-03-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127245/#review93055 --- Ship it! I couldn't find "AppData" or "Roaming" in kdelibs