openalea-incubator / caribu

Eco-physiological model of light interception by plants
Other
6 stars 11 forks source link

Sconsx not found on travis #17

Closed christian34 closed 7 years ago

christian34 commented 7 years ago

The travis build is currently failing due to import error of sconsx. any clue ?

artzet-s commented 7 years ago

Hello christian,

It's certainly due to the github version. You need to replace the follow lines in the .travis.yml :

" - pip install git+https://github.com/openalea/deploy - pip install git+https://github.com/openalea/sconsx " by

" - conda install -c openalea openalea.deploy - conda install -c openalea openalea.sconsx "

I propose this, because I reproduce the travis error on my machine with :

conda create -n test_caribu_1 python -y activate test_caribu_1 pip install git+https://github.com/openalea/deploy pip install git+https://github.com/openalea/sconsx python -c "from openalea.sconsx import config, environ" => Resulting a ImportError: No module named SCons.Script

And no error with the release version on conda :

conda create -n test_caribu_2 python -y activate test_caribu_2 conda install -c openalea openalea.deploy conda install -c openalea openalea.sconsx python -c "from openalea.sconsx import config, environ"

artzet-s commented 7 years ago

I directly submit a pull request with this modification PR #18

artzet-s commented 7 years ago

After running with this modification on travis via the PR, that failed after with :

error: [Errno 2] No such file or directory: 'build-scons/bin'

So maybe is not a good solution, you need the sconsx expert @pradal

christian34 commented 7 years ago

Cool ! unfortunately, some lib errors at install now appears : any (new) clue ?

artzet-s commented 7 years ago

resolve with PR #18