Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Kevin Ottens
On Tuesday 07 January 2014 23:30:28 John Layt wrote: > On 7 January 2014 19:49, Kevin Ottens wrote: > >> Most of the > >> dialog code from there has been merged into Qt5.2, or is planned for > >> Qt 5.3, so needs deleting. I'm also wondering if we still need our > >> own KPrintPreview dialog, the

Review Request 114908: Use add_definitions directly, instead of via _KDE4_PLATFORM_DEFINITIONS

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114908/ --- Review request for Build System and KDE Frameworks. Repository: extra-cma

Re: Review Request 114903: Do not set CMP0022

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114903/ --- (Updated Jan. 7, 2014, 11 p.m.) Status -- This change has been marke

Re: Review Request 114903: Do not set CMP0022

2014-01-07 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114903/#review47016 --- This review has been submitted with commit eb7cdf3cbdf11f22a3

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
On 07/01/14 22:34, Stephen Kelly wrote: > Alex Merry wrote: > >> On 07/01/14 18:00, Stephen Kelly wrote: >>> Alex Merry wrote: >>> Hrm. After some investigation: I'm not entirely sure. The simplest answer is "that's what qmake does" >>> >>> Please tell me you noticed that I asked about

Re: Review Request 114903: Do not set CMP0022

2014-01-07 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114903/#review47015 --- Ship it! This didn't work at all because NO_POLICY_SCOPE is n

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Alex Merry
On 07/01/14 22:30, John Layt wrote: > Running through the print dialog features, I'm don't think there is > anything left to be upstreamed. There's a couple of very minor CUPS > features in the dialog that I don't think anyone uses (mirror page, > page border, page label) that I don't really want

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Stephen Kelly
Alex Merry wrote: > On 07/01/14 18:00, Stephen Kelly wrote: >> Alex Merry wrote: >> >>> Hrm. After some investigation: I'm not entirely sure. The simplest >>> answer is "that's what qmake does" >> >> Please tell me you noticed that I asked about -U, not -D ... > > Yes. qmake actually doesn't

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Michal Humpula
On Tuesday 07 of January 2014 23:10:11 John Layt wrote: > On 7 January 2014 19:55, Albert Astals Cid wrote: > > El Dimarts, 7 de gener de 2014, a les 18:24:41, Alex Merry va escriure: > >> On 07/01/14 17:10, John Layt wrote: > >> > I've put myself down (rather obviously) for KPrintUtils. Most of

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread John Layt
On 7 January 2014 19:49, Kevin Ottens wrote: >> Most of the >> dialog code from there has been merged into Qt5.2, or is planned for >> Qt 5.3, so needs deleting. I'm also wondering if we still need our >> own KPrintPreview dialog, there was a reason in 4.0 but I can't >> remember why now and I'm

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Albert Astals Cid
El Dimarts, 7 de gener de 2014, a les 23:10:11, John Layt va escriure: > On 7 January 2014 19:55, Albert Astals Cid wrote: > > El Dimarts, 7 de gener de 2014, a les 18:24:41, Alex Merry va escriure: > >> On 07/01/14 17:10, John Layt wrote: > >> > I've put myself down (rather obviously) for KPrintU

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread John Layt
On 7 January 2014 19:55, Albert Astals Cid wrote: > El Dimarts, 7 de gener de 2014, a les 18:24:41, Alex Merry va escriure: >> On 07/01/14 17:10, John Layt wrote: >> > I've put myself down (rather obviously) for KPrintUtils. Most of the >> > dialog code from there has been merged into Qt5.2, or i

Re: Review Request 114904: Remove KDE4 magic from _SET_FANCY in KDEInstallDirs.cmake

2014-01-07 Thread David Faure
> On Jan. 7, 2014, 9:43 p.m., Alexander Neundorf wrote: > > The motivation was that if somebody had set up carefully his install dirs > > for kdelibs, he simply wanted to point a following project to the same > > CMAKE_INSTALL_PREFIX, and have all the other install dirs automatically use > > t

Re: Review Request 114904: Remove KDE4 magic from _SET_FANCY in KDEInstallDirs.cmake

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 9:43 p.m., Alexander Neundorf wrote: > > The motivation was that if somebody had set up carefully his install dirs > > for kdelibs, he simply wanted to point a following project to the same > > CMAKE_INSTALL_PREFIX, and have all the other install dirs automatically use > > t

Re: Review Request 114904: Remove KDE4 magic from _SET_FANCY in KDEInstallDirs.cmake

2014-01-07 Thread Alexander Neundorf
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114904/#review47008 --- The motivation was that if somebody had set up carefully his i

Review Request 114904: Remove KDE4 magic from _SET_FANCY in KDEInstallDirs.cmake

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114904/ --- Review request for Build System and KDE Frameworks. Repository: extra-cma

Re: Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 7:46 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 676e746b78d8b6ada47dd15fd706d0deb9996e45 by Alex Merry to branch master. > > Alexander Richardson wrote: > A bit late, but how about adding the include(GenerateExportHeader) to > KDECMakeSet

Review Request 114903: Do not set CMP0022

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114903/ --- Review request for Build System, KDE Frameworks and Stephen Kelly. Reposi

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
Fixed :P - Ursprüngliche Mail - > Am Dienstag, 7. Januar 2014, 19:57:56 schrieb Christoph Cullmann: > > Hi, > > > > I just tried to fix the naming issues. > > > > Does that try here look better > > > > http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git > > > > If yes, I wo

Re: Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Alexander Richardson
> On Jan. 7, 2014, 8:46 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 676e746b78d8b6ada47dd15fd706d0deb9996e45 by Alex Merry to branch master. A bit late, but how about adding the include(GenerateExportHeader) to KDECMakeSettings.cmake, this way the frameworks don

Re: KTextEditor & Frameworks question

2014-01-07 Thread Kevin Funk
Am Dienstag, 7. Januar 2014, 21:20:40 schrieb Kevin Funk: > Am Dienstag, 7. Januar 2014, 19:57:56 schrieb Christoph Cullmann: > > Hi, > > > > I just tried to fix the naming issues. > > > > Does that try here look better > > > > http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git > >

Re: KTextEditor & Frameworks question

2014-01-07 Thread Kevin Funk
Am Dienstag, 7. Januar 2014, 19:57:56 schrieb Christoph Cullmann: > Hi, > > I just tried to fix the naming issues. > > Does that try here look better > > http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git > > If yes, I would ask sysadmin to move to framworks, if that is ok and remo

Re: KTextEditor & Frameworks question

2014-01-07 Thread David Faure
On Tuesday 07 January 2014 21:12:39 Christoph Cullmann wrote: > Yeah, 2 => 4, all right, wanted that anyway since long. > Only the autotests/input stuff should be best left untouched. That is no > "source" in that case only input for tests, and yeah, I guess they won't > like that. OK, reverted th

Re: TODO: Reduce the mentions of KDE4 in the source code to those that are correct/needed

2014-01-07 Thread Alex Merry
On 07/01/14 20:02, Siddharth Sharma wrote: > Aha :) , Ok on it > > > On Wed, Jan 8, 2014 at 1:20 AM, David Faure > wrote: > No, this is about KDE4_* in extra-cmake-modules for instance. Or in any > CMakeLists.txt files. The task description looks like it's more abo

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
> On Tuesday 07 January 2014 20:55:19 Christoph Cullmann wrote: > > > * run astyle-kdelibs (requires some patching, so I can do it if you want) > > > > Sure, that would be nice. > > OK. Can you give a quick look at > http://www.davidfaure.fr/2014/ktexteditor-astyled.diff > and give me a green lig

Re: KTextEditor & Frameworks question

2014-01-07 Thread David Faure
On Tuesday 07 January 2014 20:55:19 Christoph Cullmann wrote: > > * run astyle-kdelibs (requires some patching, so I can do it if you want) > > Sure, that would be nice. OK. Can you give a quick look at http://www.davidfaure.fr/2014/ktexteditor-astyled.diff and give me a green light? No need to

Re: TODO: Reduce the mentions of KDE4 in the source code to those that are correct/needed

2014-01-07 Thread Siddharth Sharma
Aha :) , Ok on it On Wed, Jan 8, 2014 at 1:20 AM, David Faure wrote: > On Wednesday 08 January 2014 01:17:50 Siddharth Sharma wrote: > > Hiya Folks, > > > > I want to give a helping hand to that task, hence added my name. So if i > > understood correctly does it needs to be done with kde4suppor

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114897/ --- (Updated Jan. 7, 2014, 7:58 p.m.) Review request for Build System, KDE Fr

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
> > > > add_library (KF5TextEditor ${ktexteditor_LIB_SRCS} > > > > > > ${KTEXTEDITOR_PUBLIC_HEADERS}) > > > > > > Why pass headers to add_library? > > > > Just for automoc, without that, I get stuff like: > > > > Linking CXX shared library libKF5TextEditor.so > > CMakeFiles/KF5TextEditor.dir/vi

Re: TODO: Reduce the mentions of KDE4 in the source code to those that are correct/needed

2014-01-07 Thread David Faure
On Wednesday 08 January 2014 01:17:50 Siddharth Sharma wrote: > Hiya Folks, > > I want to give a helping hand to that task, hence added my name. So if i > understood correctly does it needs to be done with kde4support, i mean > kde4support -> kf5support ? Definitely not :) kde4support is kde4supp

Re: KTextEditor & Frameworks question

2014-01-07 Thread David Faure
> > > add_library (KF5TextEditor ${ktexteditor_LIB_SRCS} > > > > ${KTEXTEDITOR_PUBLIC_HEADERS}) > > > > Why pass headers to add_library? > > Just for automoc, without that, I get stuff like: > > Linking CXX shared library libKF5TextEditor.so > CMakeFiles/KF5TextEditor.dir/view/kateviewinternal.

TODO: Reduce the mentions of KDE4 in the source code to those that are correct/needed

2014-01-07 Thread Siddharth Sharma
Hiya Folks, I want to give a helping hand to that task, hence added my name. So if i understood correctly does it needs to be done with kde4support, i mean kde4support -> kf5support ? Regards, Siddharth Sharma ___ Kde-frameworks-devel mailing list Kde-f

Jenkins build became unstable: kwindowsystem_master_qt5 #11

2014-01-07 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 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114898/#review47000 --- This review has been submitted with commit 676e746b78d8b6ada4

Re: Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114898/ --- (Updated Jan. 7, 2014, 7:46 p.m.) Status -- This change has been mar

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
> On Tuesday 07 January 2014 19:57:56 Christoph Cullmann wrote: > > Hi, > > > > I just tried to fix the naming issues. > > > > Does that try here look better > > > > http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git > > I see a ${FooBar_HEADERS} in src/CMakeLists.txt which is used

Re: KTextEditor & Frameworks question

2014-01-07 Thread David Faure
On Tuesday 07 January 2014 19:57:56 Christoph Cullmann wrote: > Hi, > > I just tried to fix the naming issues. > > Does that try here look better > > http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git I see a ${FooBar_HEADERS} in src/CMakeLists.txt which is used but not set, should

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/#review46997 --- This review has been submitted with commit 4068592ad9aa3f2410

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/ --- (Updated Jan. 7, 2014, 7:13 p.m.) Status -- This change has been mar

[extra-cmake-modules] kde-modules: Remove custom build types

2014-01-07 Thread Alex Merry
Git commit 4068592ad9aa3f241027f6dbd6aff5f756671fd3 by Alex Merry. Committed on 03/01/2014 at 12:14. Pushed by alexmerry into branch 'master'. Remove custom build types KDECompilerSettings.cmake no longer alters CMake's built-in build types or adds its own. The "debug" build type therefore simpl

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/#review46996 --- Ship it! Nice, this also fixes some inconsistencies between c

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
Hi, I just tried to fix the naming issues. Does that try here look better http://quickgit.kde.org/?p=scratch%2Fcullmann%2Fktexteditor.git If yes, I would ask sysadmin to move to framworks, if that is ok and remove the stuff in frameworks branch of kate.git. Greetings Christoph - Ursprüng

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Albert Astals Cid
El Dimarts, 7 de gener de 2014, a les 18:24:41, Alex Merry va escriure: > On 07/01/14 17:10, John Layt wrote: > > I've put myself down (rather obviously) for KPrintUtils. Most of the > > dialog code from there has been merged into Qt5.2, or is planned for > > Qt 5.3, so needs deleting. I'm also w

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Kevin Ottens
Hello, On Tuesday 07 January 2014 18:10:10 John Layt wrote: > On 6 January 2014 07:52, Kevin Ottens wrote: > > I urge everyone, and in particular people volunteering to maintain a > > framework, to do a pass of review of our code base and APIs to modernize > > them when appropriate. It is a very

Re: KF5 Update Meeting Minutes 2014-w2

2014-01-07 Thread Albert Astals Cid
El Dimarts, 7 de gener de 2014, a les 17:28:29, Kevin Ottens va escriure: > Hello, > > As a reminder the maintainers list is available there: > http://community.kde.org/Frameworks/List Can I suggest changing "Unmaintained" to "looking for a maintainer"? It looks pretty bad that we released a the

Re: TP1 release

2014-01-07 Thread David Faure
On Sunday 05 January 2014 13:42:49 David Faure wrote: > I made and uploaded the tarballs (and zips) for the TP1 release. > > Let's give the packagers a few days, and then we'll publish and announce the > release. > > Meanwhile, no major changes in frameworks please, in case I need to redo > tarba

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
On 07/01/14 18:00, Stephen Kelly wrote: > Alex Merry wrote: > >> Hrm. After some investigation: I'm not entirely sure. The simplest >> answer is "that's what qmake does" > > Please tell me you noticed that I asked about -U, not -D ... Yes. qmake actually doesn't use -U directly; it simply set

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Alex Merry
On 07/01/14 17:10, John Layt wrote: > I've put myself down (rather obviously) for KPrintUtils. Most of the > dialog code from there has been merged into Qt5.2, or is planned for > Qt 5.3, so needs deleting. I'm also wondering if we still need our > own KPrintPreview dialog, there was a reason in

Re: KF5 Update Meeting Minutes 2014-w2

2014-01-07 Thread Alex Merry
On 07/01/14 16:28, Kevin Ottens wrote: >> * alexmerry [...] plans to adopt kimageformats and kmediaplayer; > > Alex: Both KImageFormats and KMediaPlayer are your if you adjust the wiki > page > accordingly. I've also taken KDesignerPlugin; as far as I'm aware, I'm the only person to have done

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 4:55 p.m., Stephen Kelly wrote: > > Why is -UQT_NO_EXCEPTIONS needed? > > Alex Merry wrote: > Hrm. After some investigation: I'm not entirely sure. The simplest > answer is "that's what qmake does" - defines QT_NO_EXCEPTIONS exactly when > exceptions are disabled (note

Re: Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 5:03 p.m., Stephen Kelly wrote: > > Does test code need to be compiled with the flags? I see no reason why it would. It's only really useful for libraries and plugins, as far as I'm aware, which I suspect is why CMake doesn't set it for those targets (just checked that it d

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Stephen Kelly
Alex Merry wrote: > Hrm. After some investigation: I'm not entirely sure. The simplest > answer is "that's what qmake does" Please tell me you noticed that I asked about -U, not -D ... Thanks, Steve. ___ Kde-frameworks-devel mailing list Kde-frame

Re: KF5 Update Meeting Minutes 2014-w2

2014-01-07 Thread Siddharth Sharma
Hello, I am also looking to maintain a project, if there is something where i can help. I would try to join the meeting. Regards, Siddharth Sharma On Tue, Jan 7, 2014 at 9:23 PM, Kevin Ottens wrote: > Hello everyone, > > This is the minutes of the Week 48 KF5 meeting. As usual it has been

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 4:55 p.m., Stephen Kelly wrote: > > Why is -UQT_NO_EXCEPTIONS needed? Hrm. After some investigation: I'm not entirely sure. The simplest answer is "that's what qmake does" - defines QT_NO_EXCEPTIONS exactly when exceptions are disabled (note that exceptions are disabled by

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread John Layt
On 6 January 2014 07:52, Kevin Ottens wrote: > I urge everyone, and in particular people volunteering to maintain a > framework, to do a pass of review of our code base and APIs to modernize them > when appropriate. It is a very big task, and in no way can be coordinated in > the way we've been w

Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114898/ --- Review request for Build System, KDE Frameworks and Stephen Kelly. Reposi

Re: Review Request 114898: Remove use of obsolete add_compiler_export_flags

2014-01-07 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114898/#review46985 --- Ship it! Does test code need to be compiled with the flags?

Re: Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114897/#review46984 --- Why is -UQT_NO_EXCEPTIONS needed? - Stephen Kelly On Jan. 7

Review Request 114897: Make KDE_ENABLE_EXCEPTIONS a list

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114897/ --- Review request for Build System, KDE Frameworks and Stephen Kelly. Reposi

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/ --- (Updated Jan. 7, 2014, 4:30 p.m.) Review request for Build System, KDE Fr

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 3:36 p.m., Stephen Kelly wrote: > > kde-modules/KDECompilerSettings.cmake, line 25 > > > > > > Setting CMAKE_CXX_FLAGS is not 'modern cmake'. Prefer to use > > add_compile_options instead. > >

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Stephen Kelly
> On Jan. 7, 2014, 3:36 p.m., Stephen Kelly wrote: > > kde-modules/KDECompilerSettings.cmake, line 25 > > > > > > Setting CMAKE_CXX_FLAGS is not 'modern cmake'. Prefer to use > > add_compile_options instead. > >

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 3:36 p.m., Stephen Kelly wrote: > > kde-modules/KDECompilerSettings.cmake, line 25 > > > > > > Setting CMAKE_CXX_FLAGS is not 'modern cmake'. Prefer to use > > add_compile_options instead. > >

Re: KF5 Update Meeting Minutes 2014-w2

2014-01-07 Thread Kevin Ottens
Hello, As a reminder the maintainers list is available there: http://community.kde.org/Frameworks/List People willing to maintain a framework should put their name in the corresponding column. On Tuesday 07 January 2014 16:53:30 Kevin Ottens wrote: > * agateau plans to work on kapidox, [...]

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Kevin Ottens
On Tuesday 07 January 2014 07:22:01 Kevin Ottens wrote: > On Tuesday 07 January 2014 00:45:19 Christoph Feck wrote: > > On Monday 06 January 2014 23:54:46 Kevin Ottens wrote: > > > On Monday 06 January 2014 22:26:27 Alexander Neundorf wrote: > > > > IMO something like proposing the maintainers and

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 3:36 p.m., Stephen Kelly wrote: > > kde-modules/KDECompilerSettings.cmake, line 34 > > > > > > add_compiler_export_flags() should not be used at all. > > > > http://cmake.org/gitweb?p=

KF5 Update Meeting Minutes 2014-w2

2014-01-07 Thread Kevin Ottens
Hello everyone, This is the minutes of the Week 48 KF5 meeting. As usual it has been held on #kde-devel at 4pm Paris time. Were present: afiestas, agateau, alexmerry, apol, d_ed, mck182, mgraesslin, Riddell, sandsmark, sebas, svuorela and myself. Announcement: * Happy new year! * TP1 is offi

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/#review46978 --- I didn't follow this discussion closely enough. I'll see if I

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
> On Jan. 7, 2014, 2:41 p.m., Aurélien Gâteau wrote: > > Ship It! Given that this was a fairly contentious thing on the previous RR, I'd like to have a second ship it - preferably from dfaure, ervin or skelly. - Alex --- This is an aut

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/ --- (Updated Jan. 7, 2014, 3:22 p.m.) Review request for Build System, KDE Fr

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Aurélien Gâteau
Le mardi 7 janvier 2014 11:36:34 Alex Merry a écrit : > On 06/01/14 06:52, Kevin Ottens wrote: > > I used to say that the maintenance model of kdelibs was "David Faure by > > default". It's great to have someone like David around, but at the same > > time it's delusional and dangerous to think that

Re: Review Request 114895: Guard against null QX11Info::connection()

2014-01-07 Thread Martin Gräßlin
> On Jan. 7, 2014, 3:14 p.m., Martin Gräßlin wrote: > > checking obviously makes sense, though it shouldn't be needed. There must > > be something else which is wrong here, too. > > > > Could you try what the value of WId is in these cases? I wouldn't be > > surprised if it were 0. > > > > Oh

Re: Review Request 114885: Remove custom build types

2014-01-07 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114885/#review46972 --- Ship it! Ship It! - Aurélien Gâteau On Jan. 6, 2014, 3:35

Re: Review Request 114895: Guard against null QX11Info::connection()

2014-01-07 Thread David Edmundson
> On Jan. 7, 2014, 2:14 p.m., Martin Gräßlin wrote: > > checking obviously makes sense, though it shouldn't be needed. There must > > be something else which is wrong here, too. > > > > Could you try what the value of WId is in these cases? I wouldn't be > > surprised if it were 0. > > > > Oh

Re: Review Request 114895: Guard against null QX11Info::connection()

2014-01-07 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114895/#review46967 --- checking obviously makes sense, though it shouldn't be needed.

Review Request 114895: Guard against null QX11Info::connection()

2014-01-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114895/ --- Review request for KDE Frameworks, Martin Gräßlin and Marco Martin. Repos

Re: KDE Frameworks: Moving toward 5.0 final and Governance

2014-01-07 Thread Alex Merry
On 06/01/14 06:52, Kevin Ottens wrote: > I used to say that the maintenance model of kdelibs was "David Faure by > default". It's great to have someone like David around, but at the same time > it's delusional and dangerous to think that he'll always be able to save the > day. This model has to

Re: KTextEditor & Frameworks question

2014-01-07 Thread Martin Klapetek
On Tue, Jan 7, 2014 at 9:37 AM, Christoph Cullmann wrote: > > Should the heuristic be: If there is a K in front, remove it and do stuff > like > KFooBar leads to KF5FooBar, else keep the complete FooBar name and do > KF5FooBar. > Yes, pretty much that. Cheers -- Martin Klapetek | KDE Developer

Re: KTextEditor & Frameworks question

2014-01-07 Thread Christoph Cullmann
> On Tuesday 07 January 2014 07:12:40 Christoph Cullmann wrote: > > > > I tried my luck with splitting/grafting/kdeexamples template. > > > > > > > > Could somebody take a look what ended up in the master branch of > > > > > > > > g...@git.kde.org:scratch/cullmann/ktexteditor.git > > > > > > > >

Re: KTextEditor & Frameworks question

2014-01-07 Thread David Faure
On Tuesday 07 January 2014 07:12:40 Christoph Cullmann wrote: > > > I tried my luck with splitting/grafting/kdeexamples template. > > > > > > Could somebody take a look what ended up in the master branch of > > > > > > g...@git.kde.org:scratch/cullmann/ktexteditor.git > > > > > > Any feedback we