Project

General

Profile

Feature #1120

Make build work with multi-cofiguration generators

Added by Roland Schulz over 6 years ago. Updated about 5 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
-
Target version:
Difficulty:
uncategorized
Close

Description

Running the tests from the build for multi-configuration generators doesn't work. When running ctest "-C ${CMAKE_CFG_INTDIR}" has to be added. Also ${CTEST_CONFIGURATION_TYPE} has to be added to the PATH (for each kernel,tools,...). But the escaping of set_tests_properties makes it impossible to pass in a string with unevaluated variable. Given that CTestTestfile.cmake (the file generated by add_test and read by ctest) is shared between configurations, the configuration name needs to be evaluated when ctest runs. This seems to be a known problem: http://stackoverflow.com/questions/9467510/how-to-pass-ctest-configuration-type-using-add-test-in-cmake


Related issues

Related to GROMACS - Bug #1827: cmake multi-configuration generator support partly brokenNew

History

#1 Updated by Mark Abraham over 6 years ago

What's a "configuration"? Release/Debug/Reference? What's the use case for different configuration generators?

#2 Updated by Roland Schulz over 6 years ago

Yes Release/Debug are the configurations. Some generators (e.g. Unix Makefile) work by choosing one configuration when the Makefile is generated. Others (like the Visual Studio) one create a IDE project file for each of the configurations and the configuration can be chosen in the IDE after it was generated by cmake.

#3 Updated by Rossen Apostolov about 5 years ago

So there seems to be proper support in cmake-2.8.12 (http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=07934636)but in master we are still at 2.8.8. The stackoverflow discussion suggests a hack for earlier ones. This is not critical so we could wait until we bump the cmake requirement.

#4 Updated by Teemu Murtola almost 4 years ago

  • Related to Bug #1827: cmake multi-configuration generator support partly broken added

Also available in: Atom PDF