Re: [cmake-developers] CMake Daemon blog

2016-02-14 Thread Klaim - Joël Lamotte
Nice work! I would love to help but I'm already overloaded with tons of work for now. :/ On 14 February 2016 at 13:47, Tobias Hunger wrote: > Hi Klaim, > > I spent the WE to improve the unit tests in my fork of Stephen's > cmake-daemon (which got broken when I added

[cmake-developers] [PATCH 1/2] FindwxWidgets.cmake: Fix up wxWidgets_CXXFLAGS

2016-02-14 Thread Simon Richter
This variable contains a semicolon-separated list, but a string of space-separated options is needed. As -I and -D options are separated out, this usually does not cause any trouble, unless more than one option is needed. --- Modules/FindwxWidgets.cmake | 4 1 file changed, 4 insertions(+)

[cmake-developers] [PATCH 2/2] FindwxWidgets.cmake: Resolve CygWin/MSYS paths to Windows paths

2016-02-14 Thread Simon Richter
CMake expects Windows style paths on MSYS, so use the cygpath utility to resolve them. --- Modules/FindwxWidgets.cmake | 28 1 file changed, 28 insertions(+) diff --git a/Modules/FindwxWidgets.cmake b/Modules/FindwxWidgets.cmake index 12cb1ca..ed34216 100644 ---

[cmake-developers] [PATCH 0/2] FindwxWidgets: two more msys patches

2016-02-14 Thread Simon Richter
Hi, these are two more patches from the KiCad project, more specifically Wayne Stambaugh -- all I did was separate them out, so these can't be attributed to me. These are required for KiCad to build correctly on MSYS2. Simon Simon Richter (2): FindwxWidgets.cmake:

Re: [cmake-developers] CMake Daemon blog

2016-02-14 Thread Tobias Hunger
Hi Klaim, I spent the WE to improve the unit tests in my fork of Stephen's cmake-daemon (which got broken when I added some basic protocol features earlier in the same fork). I could use some help in this task that does not involve C++ coding:-) Best Regards, Tobias Am 14.02.2016 12:58 schrieb

Re: [cmake-developers] Using CMake generated ninja file as a subninja file

2016-02-14 Thread Nicolas Desprès
Ben, I have pushed a complete version of the patch here: https://github.com/nicolasdespres/CMake/commits/ninja-output-path-prefix Since the last push, I have addressed the comments you made in your previous review (i.e. move some code to cmAlgorithm.h) and added a RunCMake.NinjaOutputPathPrefix

Re: [cmake-developers] CMake Daemon blog

2016-02-14 Thread Klaim - Joël Lamotte
I'm surprised too that there isn't more feedback around here. This seems to be a marjor potential improvement and definitely necessary if CMake need to be usable in the coming years. On 10 February 2016 at 09:42, Tobias Hunger wrote: > Hey, I did send a MR on github:-)