[cmake-developers] Initial Attempt at Green Hill MULTI IDE Generator Support

Brad King brad.king at kitware.com
Tue Nov 25 15:06:10 EST 2014


On 11/17/2014 05:16 PM, Geoffrey Viola wrote:
> I made some changes and rebased on the trunk.

Thanks.  Here are a few minor comments:

* The hunk in Modules/CMakeDetermineCompilerId.cmake is now just a
  whitespace change, so please drop it.

* Please ensure C++ sources do not have lines exceeding 79 columns.

* Please avoid trailing whitespace on source lines.

Next we need to add tests.  Since CMake won't build with this
generator we need to add tests using the generator as part of
the test suite when the needed tools are available.  Look in
Tests/CMakeLists.txt for uses of add_test_VSWinStorePhone as
an example of how this is done for another target platform.
You can either add a new test directory with dedicated code
covering capabilities of the generator, or re-use some of the
existing tests to see if they build under the new generator,
or both.

Once you get tests working, please look at submitting an
experimental build to our dashboard and posting a link to it.
You can do that from the build tree of CMake with

  ctest -C Debug -D Experimental

(or with whatever configuration you built locally).

Thanks,
-Brad


More information about the cmake-developers mailing list