Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis: add python variant #472

Merged
merged 3 commits into from
Aug 24, 2015
Merged

Conversation

cchampet
Copy link
Member

Fix #468
Fix #411

The tests are launch only when the compilation is done with gcc.
@cchampet cchampet force-pushed the travis_pythonVariant branch 8 times, most recently from 62d6432 to 84a7c08 Compare August 20, 2015 16:20
@cchampet
Copy link
Member Author

Error in python tests:
File "/home/travis/build/tuttleofx/TuttleOFX/install-tuttle/lib/python3.2/site-packages/pyTuttle/tuttle.py", line 26, in swig_import_helper
_mod = imp.load_module('_tuttle', fp, pathname, description)
ImportError: dynamic module does not define init function (init_tuttle)

@cchampet cchampet force-pushed the travis_pythonVariant branch 2 times, most recently from 109f0c4 to b8713a4 Compare August 21, 2015 08:06
@cchampet cchampet force-pushed the travis_pythonVariant branch from b8713a4 to 6d6a872 Compare August 21, 2015 09:02
@cchampet
Copy link
Member Author

CMake cannot find brewed python3 on OSX:
Homebrew/legacy-homebrew#25118

The solution could be to indicate python3 to cmake:

cmake .. -DTUTTLE_PYTHON_VERSION=3.2 -DPYTHON_EXECUTABLE=/usr/local/opt/python3/bin/python3.4 -DPYTHON_LIBRARY=/usr/local/opt/python3/bin/python3.4/lib/libpython3.4.dylib -DPYTHON_INCLUDE_DIR=/usr/local/Cellar/python3/3.4.2_1/Frameworks/Python.framework/Versions/3.4/include/python3.4m

Like we do with the homebrew formula:
https://github.com/cbenhagen/homebrew-video/blob/master/tuttleofx.rb#L60

@cchampet
Copy link
Member Author

Other problem is:
on ubuntu there is python3.2
on mac osx there is python3.4

@cchampet cchampet force-pushed the travis_pythonVariant branch from 6324977 to 0fe9320 Compare August 21, 2015 16:29
cchampet added a commit that referenced this pull request Aug 24, 2015
@cchampet cchampet merged commit 64a6bc7 into tuttleofx:develop Aug 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant