View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0012539 | CMake | (No Category) | public | 2011-10-26 04:06 | 2012-04-02 10:10 | ||||
Reporter | Jean-Christophe Fillion-Robin | ||||||||
Assigned To | David Cole | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | CMake 2.8.7 | Fixed in Version | CMake 2.8.7 | ||||||
Summary | 0012539: Expect CTestConfig.cmake in either the build or source directory | ||||||||
Description | Would it be possible to include the following topic in the release: https://github.com/jcfr/CMake/tree/check-ctestconfig-in-builddir [^] The change is documented and tests have been added: Expect CTestConfig.cmake in either the build or source directory … Provide the ability to configure CTest with setting different from the one available in the source tree by checking first if CTestConfig.cmake exists in the build tree first. The motivation is to allow build system checking out external project to test and/or package them and submit the associated results to a different dashboard than the one specified (or not) in the source of the external project. For example, the build system of Slicer can checkout, build, test and package what I will call "extension". These extensions can be developed by third party who can test and submit to their own dashboard / project. When checkout by Slicer build system, the default dashboard can be now overwritten by adding a custom CTestConfig.cmake to the build directory. And if not overwritten, it would avoid to create CTestConfig.cmake within the source checkout of the extension. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0027653) Jean-Christophe Fillion-Robin (reporter) 2011-10-26 04:08 edited on: 2011-10-26 04:09 |
From David Cole - Sep 22 2011: [...]you can override all the settings in CTestConfig already, if you set the variables at the right point in a ctest -S script (typically after the call to ctest_configure...) That would be an alternative you can consider for using with any ctest 2.8.x |
(0027659) David Cole (manager) 2011-10-26 23:44 |
Fix pushed to 'next': http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=76ecdd8d169fd2565466669875b5ed7e7c5c1963 [^] |
(0027837) David Cole (manager) 2011-11-21 15:27 edited on: 2011-11-21 15:28 |
One thing to note about this change, which I just realized after re-reviewing it with another scenario in mind... Since the code change is in the method: bool cmCTest::InitializeFromCommand(cmCTestStartCommand* command) which is only called from -S scripts that call ctest_start, this technique will only presently work with ctest -S scripts. In other words, it will NOT work with either "make Experimental" or "ctest -D Experimental" -- for those techniques to work, the information in the (still misnamed) configured file "DartConfiguration.tcl" has to point to the right CDash server... (and possibly also the CTestConfig file in the source tree) |
(0029048) David Cole (manager) 2012-04-02 10:10 |
Closing resolved issues that have not been updated in more than 4 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2011-10-26 04:06 | Jean-Christophe Fillion-Robin | New Issue | |
2011-10-26 04:08 | Jean-Christophe Fillion-Robin | Note Added: 0027653 | |
2011-10-26 04:09 | Jean-Christophe Fillion-Robin | Note Edited: 0027653 | |
2011-10-26 08:11 | David Cole | Assigned To | => David Cole |
2011-10-26 08:11 | David Cole | Status | new => assigned |
2011-10-26 08:11 | David Cole | Target Version | => CMake 2.8.7 |
2011-10-26 23:44 | David Cole | Note Added: 0027659 | |
2011-10-26 23:44 | David Cole | Status | assigned => resolved |
2011-10-26 23:44 | David Cole | Fixed in Version | => CMake 2.8.7 |
2011-10-26 23:44 | David Cole | Resolution | open => fixed |
2011-11-21 15:27 | David Cole | Note Added: 0027837 | |
2011-11-21 15:28 | David Cole | Note Edited: 0027837 | |
2012-04-02 10:10 | David Cole | Note Added: 0029048 | |
2012-04-02 10:10 | David Cole | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |