[Insight-developers] [PATCH] Send warning when ITK_DATA_ROOT is not found.

[Luis Ibanez]

Hi Matt,

Thanks for posting the patch to Gerrit.

A small improvement has been added to it:

http://review.source.kitware.com/#q,status:open+project:ITK+branch:master+topic:fail_on_itk_dataroot_not_found,n,z

Please let us know what you think.

     Thanks


           Luis


--------------------------------------------------
On Fri, Sep 3, 2010 at 10:54 AM, Matt McCormick (thewtex) <
matt at mmmccormick.com> wrote:

> If the developer forgets to set ITK_DATA_ROOT or has not downloaded the git
> submodule, ctest will start up and some XOR tests will start consuming all
> the
> system memory.
> ---
>  CMakeLists.txt |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index 7ec1165..058e8e4 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -629,6 +629,9 @@ ENDIF(HAVE_UNISTD_H)
>
>  #-----------------------------------------------------------------------------
>  # Configure the default ITK_DATA_ROOT for the location of ITK Data.
>  FIND_PATH(ITK_DATA_ROOT ITKData.readme ${ITK_SOURCE_DIR}/Testing/Data
> $ENV{ITK_DATA_ROOT})
> +IF(NOT ITK_DATA_ROOT)
> +  MESSAGE(WARNING "ITK_DATA_ROOT was not found.  ctest will not run
> properly.")
> +ENDIF()
>  MARK_AS_ADVANCED(ITK_DATA_ROOT)
>
>
>  #-----------------------------------------------------------------------------
> --
> 1.7.1
>
> _______________________________________________
> 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.html
>
> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20100904/5a59ec22/attachment.htm>