Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-13 Thread Brad King
On 10/12/2015 01:48 PM, Stephen Kelly wrote: > Subdirs: Initialize from parent before configuring. > https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7dac31b2 It seems that problem was covering another one: CMake Error: File /.../Modules/ThirdParty/GDCM/src/gdcm/Source/gdcmByteValue.cxx

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-13 Thread Stephen Kelly
Brad King wrote: > Therefore we see that CMAKE_CURRENT_SOURCE_DIR is incorrectly set. > This CMakeLists.txt file is again entered by the subdirs() command > in the parent directory. What a mess... Thanks for bisecting. I fixed this by pushing part of a branch which I was intending to start to

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-12 Thread Stephen Kelly
Brad King wrote: > Prior to this change the variable would have the right value in > the subdirectory. Now it is not set. Please take a look. This command has been documented as deprecated for a long time. I notice this command is not covered by a policy.

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-12 Thread Brad King
On 10/09/2015 10:06 AM, Kevin H. Hobbs wrote: > The ITK dashboard builds on bubbles, murron, and k450e all started > failing at the configure step on October 6. > > CMake Error at > Modules/ThirdParty/NIFTI/src/nifti/znzlib/CMakeLists.txt:18 (install): > install TARGETS given no LIBRARY

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-12 Thread Kevin H. Hobbs
On 10/12/2015 01:09 PM, Brad King wrote: > On 10/09/2015 10:06 AM, Kevin H. Hobbs wrote: >> The ITK dashboard builds on bubbles, murron, and k450e all started >> failing at the configure step on October 6. >> >> CMake Error at >> Modules/ThirdParty/NIFTI/src/nifti/znzlib/CMakeLists.txt:18

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-12 Thread Brad King
On 10/12/2015 01:48 PM, Stephen Kelly wrote: > This command has been documented as deprecated for a long time. > > I notice this command is not covered by a policy. Back when 3.0 deprecated a swath of old commands with policies we decided that subdirs() was too widely used to warn about (along

Re: [cmake-developers] ITK NIfTI broken Oct 6

2015-10-12 Thread Stephen Kelly
Brad King wrote: > On 10/12/2015 01:48 PM, Stephen Kelly wrote: >> This command has been documented as deprecated for a long time. >> >> I notice this command is not covered by a policy. > > Back when 3.0 deprecated a swath of old commands with policies > we decided that subdirs() was too widely

[cmake-developers] ITK NIfTI broken Oct 6

2015-10-09 Thread Kevin H. Hobbs
The ITK dashboard builds on bubbles, murron, and k450e all started failing at the configure step on October 6. These builds all use the development version of CMake. There were no remarkable changes in ITK that night, however there were some interesting updates to CMake. The errors are in the