On 11/01/2012 12:54 PM, Williams, Norman K wrote: > On the other hand, the Gerrit dashboard for this patch seems to indicate > we've regressed on finding include filesŠ.
This is the dependency problem you've struggled with. There is a race between building ITKIODCMTKHeaderTest1 and ITKDCMTK_ExtProject. By luck I was able to reproduce it this time around. The solution is to make header tests depend on their libraries, as with the patch below. I pushed a separate change to Gerrit for this: http://review.source.kitware.com/8282 and rebased the DCMTK changes on it. -Brad diff --git a/CMake/ITKModuleHeaderTest.cmake b/CMake/ITKModuleHeaderTest.cmake index 5b8137c..c56a995 100644 --- a/CMake/ITKModuleHeaderTest.cmake +++ b/CMake/ITKModuleHeaderTest.cmake @@ -80,6 +80,7 @@ macro( itk_module_headertest _name ) ) add_executable( ${_test_name} ${_header_test_src} ) target_link_libraries( ${_test_name} ITKCommon ) + add_dependencies(${_test_name} ${${_name}_LIBRARIES}) add_dependencies(${_name}-all ${_test_name}) math( EXPR _test_num "${_test_num} + 1" ) endforeach() _______________________________________________ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Kitware offers ITK Training Courses, for more information visit: http://kitware.com/products/protraining.php Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ Follow this link to subscribe/unsubscribe: http://www.itk.org/mailman/listinfo/insight-developers
