Sorry, I didn't send this to you guys too. I just sent it to debian-med.
Here is the patch.

On Fri, 2012-06-15 at 09:12 -0400, Bradley Lowekamp wrote:
> Hello,
> 
> 
> I have unfortunately looked at many of the other 59 failing tests for
> the 32-bit Debian sid (unstable) with gcc 4.7.0-12. I say unfortunate
> because it sounds familiar and like is should be fixable, and should
> not be punted.
> 
> 
> http://open.cdash.org/viewTest.php?onlyfailed&buildid=2363925
> 
> 
> It looks like they are all related to the same problem emitting from
> the NRRDIO library. Moreover, I recall that we have run into issue
> with NRRD being rather assertive with what is expects with NAN
> floating point handing before too. I think I have seen this error
> before.
> 
> 
> And this is a typical error message:
> 
> 
> itk::ExceptionObject (0x980deb8)
> Location: "unknown" 
> File: 
> /var/lib/jenkins/jobs/ITK-v4-Testing-32-chroot/workspace/MyTests/ITK/Modules/IO/NRRD/src/itkNrrdImageIO.cxx
> Line: 257
> Description: itk::ERROR: NrrdImageIO(0x980d738): ReadImageInformation:
> Error
> reading 
> /var/lib/jenkins/jobs/ITK-v4-Testing-32-chroot/workspace/MyTests/ITK-build/ExternalData/Testing/Data/Input/DwiCorpusCallosum.nhdr:
> [nrrd] nrrdLoad: trouble reading
> "/var/lib/jenkins/jobs/ITK-v4-Testing-32-chroot/workspace/MyTests/ITK-build/ExternalData/Testing/Data/Input/DwiCorpusCallosum.nhdr"
> [nrrd] nrrdRead: trouble
> [nrrd] _nrrdRead: sanity check FAILED: have to fix and re-compile
> [nrrd] nrrdSanity: airSanity() failed: airFPClass(AIR_QNAN,AIR_SNAN)
> wrong
> 
> 
> 
> 
> I am searching my email archive for related messages... I know we have
> had this issue before on certain platforms.
> 
> 
> Brad
> 
> ========================================================
> 
> Bradley Lowekamp  
> 
> Medical Science and Computing for
> 
> Office of High Performance Computing and Communications
> 
> National Library of Medicine 
> 
> [email protected]
> 
> 
> 
> 
> 
> 

Description: Fix test failures due to NrrdIO on i386
 Fix a bunch of failing tests, all because NrrdIO was doing some run time
 checks on NaN. The run-time check is already skipped on Apple builds and
 with Visual Studio. So now we add GCC version >= 4.7
Author: Paul Novotny <[email protected]>

--- debian-svn.orig/Modules/ThirdParty/NrrdIO/src/NrrdIO/sane.c
+++ debian-svn/Modules/ThirdParty/NrrdIO/src/NrrdIO/sane.c
@@ -107,9 +107,11 @@
 
  1) APPLE builds due to a cross-compilation problem, and
  2) Visual Studio builds for version newer than 2005 (not included)
- when building in 32bits. */
+ when building in 32bits.
+ 3) GCC 4.7 Builds when building in 32bits */
 
-#if defined(__APPLE__) || ( defined(_MSC_VER) && _MSC_VER >= 1400 ) 
+#if defined(__APPLE__) || ( defined(_MSC_VER) && _MSC_VER >= 1400 ) || \
+  ( defined(__GNUC__) && (__GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__  >= 7 )))
          /* don't compare airFP_SNAN */
 #else
          && airFP_SNAN == airFPClass_f(AIR_SNAN) 
_______________________________________________
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

Reply via email to