Re: [cmake-developers] C# support ready for review

2016-02-17 Thread Gilles Khouzam
Hi Michael, Great work, this looks really good. I have a few comments on the changes. 1. You should use the registry to find the install path for MSBuild, it should be in HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Microsoft\MSBuild with the version that you're looking for. This also bring

Re: [cmake-developers] [Qt-creator] CMake daemon for user tools

2016-02-17 Thread Milian Wolff
On Mittwoch, 17. Februar 2016 22:59:36 CET Stephen Kelly wrote: > On 02/15/2016 07:24 PM, Tobias Hunger wrote: > > Hi Dominik, > > > > Am 15.02.2016 19:01 schrieb "Dominik Haumann" > > > >: > > > 1. Wouldn't it make sense you have a developer sprint

Re: [cmake-developers] Bugfix for CPack / CPACK_INSTALL_CMAKE_PROJECTS

2016-02-17 Thread Daniel Wirtz
hey, unfortunately i wont have the time to go dig in and provide a test for that .. my work on this project literally ends in one week. we've got our own fork of cmake (https://github.com/OpenCMISS-Dependencies/CMake) that contains fix. here's a patch (based on current master

Re: [cmake-developers] CMake Daemon blog

2016-02-17 Thread Stephen Kelly
Taylor Braun-Jones wrote: > On Sat, Jan 30, 2016 at 2:06 AM, Stephen Kelly > wrote: >> I've just pushed the daemon code here: >> >> https://github.com/steveire/cmake/tree/cmake-daemon >> >> The Kate plugin should soon appear here I think: >> >>

Re: [cmake-developers] [Qt-creator] CMake daemon for user tools

2016-02-17 Thread Stephen Kelly
On 02/15/2016 07:24 PM, Tobias Hunger wrote: > > Hi Dominik, > > Am 15.02.2016 19:01 schrieb "Dominik Haumann" >: > > 1. Wouldn't it make sense you have a developer sprint ASAP for this? > > I'd be in, but I do not have the time to organize one. I could

Re: [cmake-developers] [PATCH]: CodeBlocks: improve support for different compilers

2016-02-17 Thread Alexander Neundorf
On Tuesday, February 16, 2016 10:49:24 Brad King wrote: > On 02/12/2016 03:24 AM, melven.roehrig-zoell...@dlr.de wrote: > > Just a small patch for the CodeBlocks generator > > Thanks! Applied: > > CodeBlocks: improve support for different compilers >

Re: [cmake-developers] FindCUDA.cmake regression in 3.5.0-rc2

2016-02-17 Thread Sorley, Stephen L.
I tested your fix, it's working well for my project. Thanks for the quick response! -Stephen S. -Original Message- From: Brad King [mailto:brad.k...@kitware.com] Sent: Wednesday, February 17, 2016 3:28 PM To: Sorley, Stephen L. Cc: cmake-developers@cmake.org Subject: Re:

Re: [cmake-developers] FindCUDA.cmake regression in 3.5.0-rc2

2016-02-17 Thread Brad King
On 02/17/2016 02:53 PM, Sorley, Stephen L. wrote: > FindCUDA.cmake from 3.5.0-rc2 is failing for me when creating > solution files for Visual Studio on Windows (VS 2013 update 4, > to be specific). The ninja generator on Windows is unaffected. Thanks for trying the release candidate and tracking

[cmake-developers] FindCUDA.cmake regression in 3.5.0-rc2

2016-02-17 Thread Sorley, Stephen L.
FindCUDA.cmake from 3.5.0-rc2 is failing for me when creating solution files for Visual Studio on Windows (VS 2013 update 4, to be specific). The ninja generator on Windows is unaffected. When I attempt to build a cuda target, the compile step for each .cu file fails with the following

Re: [cmake-developers] [CMake] Toolchains and CMAKE_FORCE_C_COMPILER in 3.5

2016-02-17 Thread James Crosby
> On 17 Feb 2016, at 14:33, Brad King wrote: > > One could look at adding hooks to the compiler id logic to add alternatives > provided by the toolchain file. Or, we could provide a way to set the > compiler > id but not also skip other checks. In the worst case such a

[cmake-developers] [CMake 0015978]: Visual Studio 2015 Update 2 CTP support

2016-02-17 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == https://public.kitware.com/Bug/view.php?id=15978 == Reported By:Niclas Larsén Assigned To:

Re: [cmake-developers] Bugfix for CPack / CPACK_INSTALL_CMAKE_PROJECTS

2016-02-17 Thread Brad King
On 02/16/2016 03:46 PM, Daniel Wirtz wrote: > i think i've encountered a bug where the fourth (=SubDirectory) entry of > the CPACK_INSTALL_CMAKE_PROJECTS list is simply ignored. [snip] > The offending lines of code have been committed in 2006 with > c09c3c6bfadb41bd0b43082642ce76abbf02df06 - i

Re: [cmake-developers] Pssible documentation error in install(TARGETS ) signature

2016-02-17 Thread Brad King
On 02/16/2016 05:42 PM, Daniel Wirtz wrote: > if the order of the arguments matters: update documentation? Thanks, updated: Help: Clarify install(TARGETS) INCLUDES DESTINATION option https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c978223c -Brad -- Powered by www.kitware.com Please

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

2016-02-17 Thread Brad King
On 02/16/2016 04:15 PM, Simon Richter wrote: > Hm, only problem is patch attribution here -- I've written that these > are from Wayne, and should be attributed to him (but submitting these > with his name in the From: field obviously would not have worked). IIRC if you commit with --author='...'

Re: [cmake-developers] [CMake] Toolchains and CMAKE_FORCE_C_COMPILER in 3.5

2016-02-17 Thread Brad King
Moving discussion to cmake-developers. On 02/15/2016 12:47 PM, James Crosby wrote: > I'm worried that built-in compiler detection might not be possible across > the variety of cross-compilers used. In some cases, for example, the > existence of a particular compiler, or a modified version of it,

Re: [cmake-developers] Run program/command before test

2016-02-17 Thread Roman Wüger
Hi Nils, great thanks. Regards Roman > Am 17.02.2016 um 09:09 schrieb Nils Gladitz : > >> On 17.02.2016 07:58, Roman Wüger wrote: >> Ok thanks >> >> Would it be an opinion to have a COMMAND parameter for the add_test function >> like execute_process where every

Re: [cmake-developers] Run program/command before test

2016-02-17 Thread Nils Gladitz
On 17.02.2016 07:58, Roman Wüger wrote: Ok thanks Would it be an opinion to have a COMMAND parameter for the add_test function like execute_process where every COMMAND must return the exit code and this would be ored? add_test(MyTest COMMAND myserver --port 80