[Insight-developers] Re: WrapITK : ImageCompare ..

[Karthik Krishnan]

Gaetan Lehmann wrote:

> i Karthik,
>
> Le Fri, 08 Sep 2006 19:44:43 +0200, Karthik Krishnan  
> <Karthik.Krishnan at kitware.com 
> <http://www.itk.org/mailman/listinfo/insight-developers>> a écrit:
>
>> / Also, why do I get a cache variable for IMAGE_COMPARE when I build ITK
>
> />/ and where do I get this executable from ?  The code, I can see is in
> />/ Code/IO and the executable should can be specified internally. Is 
> this
> />/ to handle paths for multi-configuration build systems like
> />/ Windows/XCode, which determine the executable path at build time ? 
> There
> />/ is a workaround for that. Let me know.
> /
> I have added it to be sure to use the version built with ITK for the  
> tests, while still being able to build WrapITK outside ITK. If there 
> is a  better way to do that, please fix :-)
> I didn't thought to multi-configurations case, and even think that 
> this  var will be wrong in that case.
>  
>
This is a hard one to fix..  Brad ??

I can't configure ITK right now with WrapITK ON cause it complains that 
IMAGE_COMPARE executable is not found..

Of course, ImageCompare.cxx its within the sources, and even if we did 
build it within ITK, we'd have to pass the right executable path to your 
test driver.. This isn't an issue on Linux, but on multi-configuration 
builds like Windows/Xcode, which determine the executable path at build 
time, you can do that from C++ code :

int testdriver() {
.....
 std::string  = 
itksys::SystemTools::ConvertToOutputPath(ImageCompareExecutableIn);
#if defined(_WIN32) && defined(CMAKE_INTDIR)
 itksys::SystemTools::ReplaceString(ImageCompareExecutable, "$(OutDir)", 
CMAKE_INTDIR);
#endif
.....
}

---------------------------

Now in your case, you have a tcl/Python ( 
http://www.itk.org/cgi-bin/viewcvs.cgi/Wrapping/WrapITK/Tcl/Tests/tclTestDriver.tcl.in?annotate=1.1&root=Insight 
) test driver, not a cxx test driver .... That isn't even compiled... So 
I have no idea how to determine a path that's determined only at build 
time.

Any way you can drive that test from a Cxx code ?

--------------------------

Thanks
-karthik

> Please let me know if there is a better way to implement that. If 
> not,  I'll test and fix the internal vars I have added.
>
> Thanks,
>
> Gaetan
>
>
>> /
>
> />/ ---------
> />/ -karthik
> />/
> />/
> />/
> />/ Gaetan Lehmann wrote:
> />/
> />>/
> />>/ What is the status on windows plateform ? I don't find a windows 
> build
> />>/ on  the dashboard, but I have surely missed it.
> />>/ If it (mostly) work on windows, then I can't see any important 
> change
> />>/ to  do :-)
> />>/
> />>/ Gaetan
> />>/
> />>/
> />>/ On Fri, 08 Sep 2006 16:01:10 +0200, Luis Ibanez
> />>/ <luis.ibanez at kitware.com 
> <http://www.itk.org/mailman/listinfo/insight-developers>>  wrote:
> />>/
> />>>/
> />>>/ Hi Gaetan,
> />>>/
> />>>/ Do you see the need for any major changes
> />>>/ related to the integration of WrapITK ?
> />>>/
> />>>/ Could we consider to tag the CVS repository
> />>>/ with the "post" integration tag ?
> />>>/
> />>>/
> />>>/ Please let us know,
> />>>/
> />>>/
> />>>/
> />>>/     Thanks
> />>>/
> />>>/
> />>>/       Luis
> />>>/
> />>>/
> />>/
> />>/
> />>/
> /
>
>
> -- 
> Gaëtan Lehmann
> Biologie du Développement et de la Reproduction
> INRA de Jouy-en-Josas (France)
> tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
> http://voxel.jouy.inra.fr
>
>