Include ExternalData conditionally

[jcfr]

See 3b3b8c4#commitcomment-2784641

By applying the method reported in the comment, Slicer will be ensured to include its own version instead of the one bundled with BRAINS.

Waiting that all of our projects depends strictly on CMake 2.8.11, it allows the ExternalData.cmake within Slicer to be different ...

[jcfr]

Would be great to also address this issue 👍 Thanks

[hjmjohnson]

JC,

Where did you get that version of ExternalData.cmake. It does not match the version that is available in ITK. I want to try to keep all copies of ExternalData.cmake consistent across DTIPrep, BRAINSTools, ANTS, ITK, DTIAtlasBuilder, etc....

Is your version compatible with ITK?

Hans

[jcfr]

This is the official version now bundled within CMake 2.8.11.
See
http://cmake.org/gitweb?p=cmake.git;a=blob;f=Modules/ExternalData.cmake;h=187f408ffd5e43a9225d72e645832d61261987fa;hb=HEAD
Hth
Jc

On Wed, Mar 27, 2013 at 2:35 PM, hjmjohnson notifications@github.comwrote:

JC,

Where did you get that version of ExternalData.cmake. It does not match
the version that is available in ITK. I want to try to keep all copies of
ExternalData.cmake consistent across DTIPrep, BRAINSTools, ANTS, ITK,
DTIAtlasBuilder, etc....

Is your version compatible with ITK?

Hans

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/3#issuecomment-15544021
.

+1 919 869 8849

[hjmjohnson]

diff ~/src/Slicer/CMake/ExternalData.cmake ~/src/BRAINSTools/CMake/ExternalData.cmake

Shows that BRAINSTools and Slicer have identical ExternalData.cmake files.