[Cmake-commits] CMake branch, master, updated. v3.9.1-591-g6285fe0

2017-08-25 Thread Kitware Robot
_VERSION_MINOR 9) -set(CMake_VERSION_PATCH 20170825) +set(CMake_VERSION_PATCH 20170826) #set(CMake_VERSION_RC 1) --- Summary of changes: Source/CMakeVersion.cmake |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/

Re: [CMake] CMake + Gradle for Android

2017-08-25 Thread Jom O'Fisher
Hi again Robert, Would you be able to give me an estimate of how many APK projects you have, roughly which open source projects you reference via CMake add_subdirectories, and whether you have any variants beyond the default Debug and Release? If possible I'd like to approximate your project

Re: [CMake] CMake + Gradle for Android

2017-08-25 Thread Jom O'Fisher
Targets are specified per-Variation so they need to go under the variation-specific section. Probably something like this: defaultConfig { externalNativeBuild { cmake { targets "library1",

Re: [CMake] CMake + Gradle for Android

2017-08-25 Thread Robert Dailey
By the way when I try to use "targets", I get a failure. Basically Gradle doesn't recognize that keyword. I tried singular form as well ("target"), no luck. I'm running canary build of everything possible. What am I missing? On Wed, Aug 23, 2017 at 4:20 PM, Jom O'Fisher

Re: [cmake-developers] Dependency graph in server mode?

2017-08-25 Thread Tobias Hunger
Hi Eric, On Fri, Aug 25, 2017 at 3:53 PM, Eric Noulard wrote: > Is it possible to get the dependency graph one can get with the --graphviz > option when running in server mode. Not yet:-) > From my understanding the current "codeModel" message does not contain such >

Re: [cmake-developers] Should configuration package files define module package variables?

2017-08-25 Thread Robert Dailey
Doh, forgot the links I intended to reference in my original email: [1]: https://cmake.org/cmake/help/v3.6/command/find_package.html [2]: https://cmake.org/cmake/help/v3.6/manual/cmake-packages.7.html#creating-packages [3]: https://cmake.org/cmake/help/v3.6/module/CMakePackageConfigHelpers.html

Re: [CMake] Should configuration package files define module package variables?

2017-08-25 Thread Robert Dailey
Doh, forgot the links I intended to reference in my original email: [1]: https://cmake.org/cmake/help/v3.6/command/find_package.html [2]: https://cmake.org/cmake/help/v3.6/manual/cmake-packages.7.html#creating-packages [3]: https://cmake.org/cmake/help/v3.6/module/CMakePackageConfigHelpers.html

[cmake-developers] Should configuration package files define module package variables?

2017-08-25 Thread Robert Dailey
So I've been studying the find_package[1] and "creating packages"[2] documentation, as well as the CMakePackageConfigHelpers[3] page. Based on the current offerings of configuration packages, I do not understand the need for the relocatable config.cmake file when all it really contains is:

[CMake] Should configuration package files define module package variables?

2017-08-25 Thread Robert Dailey
So I've been studying the find_package[1] and "creating packages"[2] documentation, as well as the CMakePackageConfigHelpers[3] page. Based on the current offerings of configuration packages, I do not understand the need for the relocatable config.cmake file when all it really contains is:

Re: [CMake] Please update the documentation for execute_process

2017-08-25 Thread Andreas Naumann
Am 25.08.2017 um 16:12 schrieb Jeffrey Walton: Below is a typical case for us (https://github.com/weidai11/cryptopp/blob/master/CMakeLists.txt). If it looks pretty shitty, it probably is. That's the best we have been able to come up with based on the documentation. Below is the example code. We

[Cmake-commits] CMake branch, master, updated. v3.9.1-588-g53305ce

2017-08-25 Thread Kitware Robot
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "CMake". The branch, master has been updated via 53305ce5b067c4feaa91ffe0cc82b740af525b3f (commit) via

[Cmake-commits] CMake branch, master, updated. v3.9.1-586-ga749fcb

2017-08-25 Thread Kitware Robot
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "CMake". The branch, master has been updated via a749fcb3d0f788b85be0531ffe7e7ec2aaf7f9fa (commit) via

Re: [CMake] Please update the documentation for execute_process

2017-08-25 Thread Jeffrey Walton
> Below is a typical case for us > (https://github.com/weidai11/cryptopp/blob/master/CMakeLists.txt). If > it looks pretty shitty, it probably is. That's the best we have been > able to come up with based on the documentation. Below is the example code. We don't know whether it should be:

[CMake] Please update the documentation for execute_process

2017-08-25 Thread Jeffrey Walton
The documentation for execute_process has some room for improvement. We recently got burned by a problem that has existed since at least 2011: https://stackoverflow.com/q/6797395/608639 . Once we learned the problem we could research a bit. The problem cost us over 8 man hours when it should not

[cmake-developers] Dependency graph in server mode?

2017-08-25 Thread Eric Noulard
Hi there, Is it possible to get the dependency graph one can get with the --graphviz option when running in server mode. >From my understanding the current "codeModel" message does not contain such information. Am I right? If I wanted to add the dependency information would it be better to

Re: [CMake] 'cmake.exe -G Ninja' doesn't work for VS2017 with cmake ver 3.9.1

2017-08-25 Thread Masaru Tsuchiyama
Hello I confirmed this is fixed. I used 9538d22d955a0b101548019003f2d5c7ba833d77 for the check. Brad King wrote: On 08/23/2017 05:07 AM, masaru tsuchiyama wrote: It seems your coworker can reproduce it and you can use the PC tomorrow. So I don't need to debug it, right? Correct, thanks.