This repository has been archived by the owner. It is now read-only.

InsightToolkit: Added support for python wrapping ('--python') - still experimental #16918

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants

dloeckx commented Jan 6, 2013

Added support for python wrapping, yet did only test on my system. The wrapping causes some errors normally, because of some older dependent packages (gccxml and cswig). These can be alleviated by using ansi-C and setting gccxml enviroment flags. I also included the module ITKVtkGlue, to connect ITK and VTK.

This is my first time, so please don't blame me too much if I am doing things wrong.

Dirk Loeckx Dirk Loeckx
Added support for python wrapping ('--python') - still experimental
Added support for python wrapping, yet did only test on my system. The wrapping causes some errors normally, because of some older dependent packages (gccxml and cswig). These can be alleviated by using ansi-C and setting gccxml enviroment flags. I also included the module ITKVtkGlue, to connect ITK and VTK.
Contributor

adamv commented Jan 6, 2013

Note, please mention the formula name in the commit message / pull request.

Can't assume 64-bit.

Contributor

samueljohn commented Jan 9, 2013

Cool, I will test this formula...

dloeckx commented Jan 9, 2013

Thanks. Please let me know if you encounter any issues.

Contributor

samueljohn commented Jan 10, 2013

Looks like ITK depends_on 'vtk', right?

brew install insighttoolkit --python --examples -vd
[snip]
Make Error at /homebrew/Cellar/cmake/2.8.10.1/share/cmake/Modules/FindVTK.cmake:135 (message):
  VTK not found.  Set the VTK_DIR cmake cache entry to the directory
Contributor

samueljohn commented Jan 10, 2013

Now getting another build error. Any clue?

https://gist.github.com/4503862

Contributor

samueljohn commented Jan 10, 2013

I am on 10.8.2 with Xcode-only install.

+
+ # These environment variables are necessary for gccxml to work properly
+ # llvm-g++ will usually be in this location, do we have to check?
+ ENV['GCCXML_COMPILER']=#{ENV.cxx}
@dloeckx

dloeckx Jan 31, 2013

Apparently #{ENV.cxx} does not translate to the correct compiler. A workaround is to replace this line with
ENV['GCCXML_COMPILER']='/usr/bin/llvm-g++-4.2'

@MikeMcQuaid

MikeMcQuaid Jan 31, 2013

Owner

It should? What makes you think it isn't correct?

@samueljohn

samueljohn Jan 31, 2013

Contributor

When I check in brew irb, ENV.cxx returns "c++". Does this formula (really) need a fails_with :clang block? Would be sad...

This syntax is wrong; need to quote this. But actually you don't need to use interpolation at all, just use ENV.cxx.

Contributor

iMichka commented Apr 29, 2013

Hi

I just tested your formula, it fails to build : https://gist.github.com/iMichka/5483868
I tried also with the newer version of itk (4.3.2), the error is different : https://gist.github.com/iMichka/5483885

Contributor

adamv commented Apr 30, 2013

Closing since this doesn't work for us; but would review and updated request.

@adamv adamv closed this Apr 30, 2013

@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.