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

2015-07-15 Thread Robert Goulet
Filimonov Sent: Wednesday, July 15, 2015 10:14 AM To: Brad King Cc: cmake-developers@cmake.org Subject: Re: [cmake-developers] [PATCH] Extended Nsight Tegra support for CMake Thanks, Brad. As usual, we're open for community feedback on Nsight Tegra project generator. -Mikhail -Original Message

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

2015-07-15 Thread Mikhail Filimonov
: [cmake-developers] [PATCH] Extended Nsight Tegra support for CMake On 07/13/2015 12:44 PM, Mikhail Filimonov wrote: I've improved the patch according to the Brad's and Dmitry's comments - namely: 1. Improved the documentation for a few target properties. 2. Removed the explicit toolchain file

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

2015-07-15 Thread Brad King
On 07/13/2015 12:44 PM, Mikhail Filimonov wrote: I've improved the patch according to the Brad's and Dmitry's comments - namely: 1. Improved the documentation for a few target properties. 2. Removed the explicit toolchain file for Nsight Tegra generator test. 3. Improved the generator

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

2015-07-13 Thread Mikhail Filimonov
. Thanks, -Mikhail -Original Message- From: cmake-developers [mailto:cmake-developers-boun...@cmake.org] On Behalf Of Mikhail Filimonov Sent: 9 июля 2015 г. 18:43 To: Brad King Cc: cmake-developers@cmake.org Subject: Re: [cmake-developers] [PATCH] Extended Nsight Tegra support for CMake

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

2015-07-13 Thread Dmitry Polyanitsa
I've reviewed the patch and tried it locally - looks good. -Dmitry -Original Message- From: Mikhail Filimonov Sent: Monday, July 13, 2015 7:45 PM To: cmake-developers@cmake.org Subject: RE: [cmake-developers] [PATCH] Extended Nsight Tegra support for CMake I've improved the patch

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

2015-07-09 Thread 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 +

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

2015-07-08 Thread Mikhail Filimonov
Hi, all! 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. The patch adds the following new Target properties: A new Target properties that maps to all Configuration PropertyGroups for each