[cmake-developers] [PATCH] Extended Nsight Tegra support for CMake

[Brad King]

On 07/08/2015 11:21 AM, Mikhail Filimonov wrote:
> I've extended the Nsight Tegra project generator in CMake and added a bunch
> of properties with the backing variables to fine-tune the generated projects.

Thanks!  Here are some comments.

> -        --build-options -DCMAKE_SYSTEM_NAME=Android
> +        --build-options -DCMAKE_SYSTEM_NAME=Android -DCMAKE_TOOLCHAIN_FILE= "${CMake_SOURCE_DIR}/Tests/VSNsightTegra/NsightTegraToolchain.cmake"
[snip]
> +# CMake toolchain file used for the generation of Nsight Tegra project files
> +
> +set(CMAKE_GENERATOR_TOOLSET clang-3.5)
> +set(CMAKE_SYSTEM_NAME Android)

We'd like to keep testing the default behavior without an explicit
toolchain file.  Also we need to be independent of the toolsets
that happen to be available on the system where the test runs,
so we should avoid hard-coding a toolset name if possible.  Is
this part of the patch needed to test everything else?  If so,
please look at adding more test cases for the new combinations
(which can just be more builds of the same test source tree).

> +set_property(TARGET twolib-second APPEND PROPERTY ANDROID_NATIVE_LIB_DIRECTORIES c:\\wrk c:\\wrk\\cmake)

The test should not name directories outside control of the test
suite.  Please name directories in the test source tree or something.

Also, in Source/* C++ sources please wrap lines to keep them <= 79
columns.

Thanks,
-Brad