[CMake] CMake updates timestamp when there are missing files.

2016-07-26 Thread Golebiewski, Jakub
Hi, I've noticed a weird (but maybe expected) behavior. My colleague forgot to edit CMakeLists.txt file for a library from which he deleted a .cpp file. As expected cmake failed with an error message during build. But next build after that passed without issues. CMake didn't run even though

[Cmake-commits] CMake branch, master, updated. v3.6.1-470-g73c618b

2016-07-26 Thread Kitware Robot
_VERSION_MINOR 6) -set(CMake_VERSION_PATCH 20160726) +set(CMake_VERSION_PATCH 20160727) #set(CMake_VERSION_RC 1) --- Summary of changes: Source/CMakeVersion.cmake |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/

Re: [cmake-developers] cmake -E capabilities [attempt 2]

2016-07-26 Thread Stephen Kelly
Tobias Hunger wrote: > Did anyone find some time for a review yet? Hi Tobias, I had a look through this this evening. Thanks for working on this. The commit adding the functionality at the end looks much better after the extra generator refactoring. Here are some review notes: * That

Re: [CMake] XCode generator and .metal shader files

2016-07-26 Thread Gregor Jasny via CMake
Hello, On 22/07/16 23:07, Victor Rykov wrote: > After adding a .metal file to a target via add_executable the metal file > does not get pushed to the generated project`s Compile Sources queue and > subsequently does not compile during project build unless I add the file to > the queue manually

Re: [cmake-developers] Autogen subdirectories patches

2016-07-26 Thread Stephen Kelly
Sebastian Holtermann wrote: >> It might be better to just use a NULL sentinel at the end of the array. >> If not, the extra spaces here should go away. > > I have changed that to use a NULL sentinel. > std::array would be the best solution IMO but it is not allowed, is it? You could use

Re: [cmake-developers] Autogen subdirectories patches

2016-07-26 Thread Sebastian Holtermann
Thanks for the feedback. I have prepared an other approach which uses checksum suffixes instead of nested directory generation. This was tested on my Debian/amd64/Makefile setup. Interesting; I like it. Might be worthwhile to reuse it for the .o files in the future as well (since these can

Re: [cmake-developers] Autogen subdirectories patches

2016-07-26 Thread Ben Boeckel
On Tue, Jul 26, 2016 at 17:31:00 +0200, Sebastian Holtermann wrote: > I have prepared an other approach which uses checksum suffixes > instead of nested directory generation. > This was tested on my Debian/amd64/Makefile setup. Interesting; I like it. Might be worthwhile to reuse it for the .o

Re: [cmake-developers] Autogen subdirectories patches

2016-07-26 Thread Sebastian Holtermann
Am 21.07.2016 um 17:40 schrieb Brad King: On 04/22/2016 09:37 AM, Brad King wrote: The rest of the changes are now in 'master'. Thanks for working on this! Unfortunately I've had to revert some of these changes due to regressions they cause. See issue here:

Re: [cmake-developers] Question over language bindings / daemon

2016-07-26 Thread Charles Huet
Hi Rick, this is something I played with a bit, and while it is completely possible to make your own little python binding, it will not be merged into CMake, which means you will have to maintain it on your side. Also, you will need to write the whole interface yourself, as the current CMake lib

Re: [cmake-developers] cmake -E capabilities [attempt 2]

2016-07-26 Thread Tobias Hunger
Did anyone find some time for a review yet? Best regards, Tobias Am 22.07.2016 01:37 schrieb "Tobias Hunger" : > Hello fellow developers, > > https://github.com/hunger/CMake/commits/cmake-capabilities > > has my attempt to get "cmake -E capabilities" merged -- in