Re: Review Request 115442: Change NET* classes and KWindoInfo/KWindowSystem to use declared flag types

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115442/ --- (Updated Feb. 4, 2014, 7:52 a.m.) Review request for KDE Frameworks. Ch

Re: Review Request 115442: Change NET* classes and KWindoInfo/KWindowSystem to use declared flag types

2014-02-03 Thread Martin Gräßlin
> On Feb. 3, 2014, 9:36 a.m., Martin Gräßlin wrote: > > src/netwm.h, lines 1426-1430 > > > > > > This method should also get adjusted but as it's virtual it will be a > > true API break. > > > > What's

Re: Review Request 115434: Deprecate KWindowSystem::windowInfo

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115434/#review48885 --- This review has been submitted with commit 31168280d47aeaff19

Re: Review Request 115434: Deprecate KWindowSystem::windowInfo

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115434/ --- (Updated Feb. 4, 2014, 6:35 a.m.) Status -- This change has been mar

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Martin Gräßlin
> On Feb. 3, 2014, 9:07 p.m., Martin Gräßlin wrote: > > src/kstartupinfo.cpp, lines 1250-1254 > > > > > > why is this introduced again? > > Alexander Richardson wrote: > NET::OnAllDesktops does not exist if

Review Request 115458: Include netwm_def.h in kstartupinfo.cpp

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115458/ --- Review request for KDE Frameworks and Alexander Richardson. Repository: k

Re: Review Request 115378: Resolve some FIXMEs related to MSVC and Intel

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115378/#review48880 --- kde-modules/KDECompilerSettings.cmake

Re: Review Request 115419: Split KCompletionMatches class into files of its own

2014-02-03 Thread David Gil Oliva
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115419/ --- (Updated Feb. 4, 2014, 12:36 a.m.) Review request for KDE Frameworks. C

Re: Review Request 115418: Hide private methods of KCompletion into d-pointer

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115418/#review48879 --- This review has been submitted with commit 0efcbfadba9c2a6738

Re: Review Request 115418: Hide private methods of KCompletion into d-pointer

2014-02-03 Thread David Gil Oliva
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115418/ --- (Updated Feb. 4, 2014, 12:30 a.m.) Status -- This change has been ma

Review Request 115457: Allow compiling KJS on windows

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115457/ --- Review request for KDE Frameworks and kdewin. Repository: kjs Descripti

Re: Review Request 115343: Fix msvc build of solid

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115343/ --- (Updated Feb. 4, 2014, 1:04 a.m.) Review request for KDE Frameworks. Re

Review Request 115456: Fix MSVC libc linkage for debug builds

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115456/ --- Review request for Build System, Extra Cmake Modules, KDE Frameworks, and

Re: Review Request 115343: Fix msvc build of solid

2014-02-03 Thread Alex Merry
> On Jan. 31, 2014, 7:41 p.m., Alex Merry wrote: > > src/solid/networking_win.cpp, lines 116-117 > > > > > > It's odd that this should be needed. Do you have any way of getting > > verbose output? > > (http:/

Re: Review Request 115395: Also pass -fno-exceptions when building with clang

2014-02-03 Thread Raphael Kubo da Costa
> On Feb. 3, 2014, 8:07 p.m., Raphael Kubo da Costa wrote: > > Please see the big comment below the elseif line, the link to the > > kde-core-devel and > > http://lists.kde.org/?l=kde-core-devel&m=138244424421211&w=2: the issue > > here is that if you pass -fno-exceptions to clang you need to

Re: Review Request 115203: Allow compiling with MSVC

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115203/ --- (Updated Feb. 3, 2014, 10:25 p.m.) Status -- This change has been ma

Re: Review Request 115203: Allow compiling with MSVC

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115203/#review48872 --- This review has been submitted with commit ad3dc08b0892a52108

Re: Review Request 115203: Allow compiling with MSVC

2014-02-03 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115203/#review48869 --- Ship it! Just one thing, then it can go in. src/kcrash.cpp

Re: Review Request 115343: Fix msvc build of solid

2014-02-03 Thread Alexander Richardson
> On Jan. 31, 2014, 8:41 p.m., Alex Merry wrote: > > src/solid/networking_win.cpp, lines 116-117 > > > > > > It's odd that this should be needed. Do you have any way of getting > > verbose output? > > (http:/

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Alexander Richardson
> On Feb. 3, 2014, 9:07 p.m., Martin Gräßlin wrote: > > src/kstartupinfo.cpp, lines 1250-1254 > > > > > > why is this introduced again? > > Alexander Richardson wrote: > NET::OnAllDesktops does not exist if

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115450/ --- (Updated Feb. 3, 2014, 9:24 p.m.) Status -- This change has been dis

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Martin Gräßlin
> On Feb. 3, 2014, 9:07 p.m., Martin Gräßlin wrote: > > src/kstartupinfo.cpp, lines 1250-1254 > > > > > > why is this introduced again? > > Alexander Richardson wrote: > NET::OnAllDesktops does not exist if

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Alexander Richardson
> On Feb. 3, 2014, 9:07 p.m., Martin Gräßlin wrote: > > src/kstartupinfo.cpp, lines 1250-1254 > > > > > > why is this introduced again? NET::OnAllDesktops does not exist if KWINDOWSYSTEM_HAVE_X11 is false. I do

Re: Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115450/#review48854 --- src/kstartupinfo.cpp

Re: Review Request 115344: With MSVC M_PI only gets defined if _USE_MATH_DEFINES is defined

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115344/#review48852 --- This review has been submitted with commit 3278a78a4a7af534e4

Re: Review Request 115344: With MSVC M_PI only gets defined if _USE_MATH_DEFINES is defined

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115344/ --- (Updated Feb. 3, 2014, 7:46 p.m.) Status -- This change has been mar

Re: Review Request 115395: Also pass -fno-exceptions when building with clang

2014-02-03 Thread Alexander Richardson
> On Feb. 3, 2014, 7:07 p.m., Raphael Kubo da Costa wrote: > > Please see the big comment below the elseif line, the link to the > > kde-core-devel and > > http://lists.kde.org/?l=kde-core-devel&m=138244424421211&w=2: the issue > > here is that if you pass -fno-exceptions to clang you need to

Review Request 115450: Fix build without KWINDOWSYSTEM_HAVE_X11

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115450/ --- Review request for KDE Frameworks. Repository: kwindowsystem Descriptio

Re: Review Request 115203: Allow compiling with MSVC

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115203/ --- (Updated Feb. 3, 2014, 7:43 p.m.) Review request for KDE Frameworks. Ch

Jenkins build is back to stable : kimageformats_master_qt5 #21

2014-02-03 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 115345: Fix kimageformats build with MSVC

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115345/#review48849 --- This review has been submitted with commit a533bcd4185026964e

Re: Review Request 115345: Fix kimageformats build with MSVC

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115345/ --- (Updated Feb. 3, 2014, 6:24 p.m.) Status -- This change has been mar

Re: Review Request 115345: Fix kimageformats build with MSVC

2014-02-03 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115345/ --- (Updated Feb. 3, 2014, 6:24 p.m.) Status -- This change has been mar

Re: Review Request 115345: Fix kimageformats build with MSVC

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115345/#review48848 --- This review has been submitted with commit 63705e373e06cb524b

Re: Review Request 115395: Also pass -fno-exceptions when building with clang

2014-02-03 Thread Alex Merry
> On Feb. 3, 2014, 6:07 p.m., Raphael Kubo da Costa wrote: > > Please see the big comment below the elseif line, the link to the > > kde-core-devel and > > http://lists.kde.org/?l=kde-core-devel&m=138244424421211&w=2: the issue > > here is that if you pass -fno-exceptions to clang you need to

Re: Review Request 115395: Also pass -fno-exceptions when building with clang

2014-02-03 Thread Raphael Kubo da Costa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115395/#review48846 --- Please see the big comment below the elseif line, the link to

Jenkins build became unstable: kimageformats_master_qt5 #18

2014-02-03 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 115430: Add simple autotests

2014-02-03 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115430/ --- (Updated Feb. 3, 2014, 3:03 p.m.) Status -- This change has been mar

Re: Review Request 115430: Add simple autotests

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115430/#review48831 --- This review has been submitted with commit f0f6d14367f85fb8e8

Re: Review Request 115440: Rename KDE4_SERIALIZE_TOOL to KDOCTOOLS_SERIALIZE_TOOL And Add Setter

2014-02-03 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115440/#review48830 --- Ship it! If it is worth to move it to another, more generic p

Re: Review Request 115430: Add simple autotests

2014-02-03 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115430/#review48829 --- Ship it! Can't really test it locally since I can't find a wa

Re: Review Request 115430: Add simple autotests

2014-02-03 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115430/ --- (Updated Feb. 3, 2014, 1:12 p.m.) Review request for KDE Frameworks and A

Re: Review Request 115418: Hide private methods of KCompletion into d-pointer

2014-02-03 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115418/#review48822 --- Ship it! There is a small conflict with KCompletionPrivate de

Re: Review Request 115430: Add simple autotests

2014-02-03 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115430/#review48821 --- autotests/readtest.cpp

Re: Review Request 115434: Deprecate KWindowSystem::windowInfo

2014-02-03 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115434/#review48820 --- Ship it! Ship It! - Aurélien Gâteau On Feb. 2, 2014, 12:37

Re: Review Request 115336: Change syntax of kservice_desktop_to_json

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115336/#review48819 --- This review has been submitted with commit 594664a3169561fe15

Re: Review Request 115442: Change NET* classes and KWindoInfo/KWindowSystem to use declared flag types

2014-02-03 Thread Aurélien Gâteau
> On Feb. 3, 2014, 9:36 a.m., Martin Gräßlin wrote: > > src/netwm.h, lines 1426-1430 > > > > > > This method should also get adjusted but as it's virtual it will be a > > true API break. > > > > What's

Re: Review Request 115336: Change syntax of kservice_desktop_to_json

2014-02-03 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115336/ --- (Updated Feb. 3, 2014, 12:29 p.m.) Status -- This change has been ma

Re: Review Request 115421: Clean up the CMakeLists.txt files

2014-02-03 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115421/#review48817 --- This review has been submitted with commit 45d0fb6ba80ea8110f

Re: Review Request 115421: Clean up the CMakeLists.txt files

2014-02-03 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115421/ --- (Updated Feb. 3, 2014, 12:26 p.m.) Status -- This change has been ma

Re: Review Request 115421: Clean up the CMakeLists.txt files

2014-02-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115421/#review48815 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 1, 2014, 1:

Re: Review Request 115336: Change syntax of kservice_desktop_to_json

2014-02-03 Thread Aurélien Gâteau
> On Jan. 31, 2014, 8:15 p.m., Alex Merry wrote: > > There was an optional jsonfile argument; I guess you checked nothing was > > using it? Because anything that was will be broken by this. > > > > That said, I have no objection to breaking source compatibility of > > something that hasn't ev

Re: kf5 alpha 1 : modules, versions

2014-02-03 Thread Kevin Ottens
On Monday 03 February 2014 10:17:49 David Faure wrote: > Any new module that should be added to this release, compared to TP1? > > Should I include attica? Since this question keeps popping up, let's integrate it. It should also be added to the list: http://community.kde.org/Frameworks/List Als

kf5 alpha 1 : modules, versions

2014-02-03 Thread David Faure
Any new module that should be added to this release, compared to TP1? Should I include attica? Any version number that should be upgraded in the modules themselves? I realize now that it's all called 5.0.0 everywhere already. The packages are properly numbered, but not the cmake variable conta

Re: Review Request 115442: Change NET* classes and KWindoInfo/KWindowSystem to use declared flag types

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115442/#review48791 --- src/netwm.h

Review Request 115442: Change NET* classes and KWindoInfo/KWindowSystem to use declared flag types

2014-02-03 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115442/ --- Review request for KDE Frameworks. Repository: kwindowsystem Descriptio

Review Request 115440: Rename KDE4_SERIALIZE_TOOL to KDOCTOOLS_SERIALIZE_TOOL And Add Setter

2014-02-03 Thread David Narváez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115440/ --- Review request for KDE Frameworks and Luigi Toscano. Repository: kdoctool