search

SyneRBI / SyneRBI_VM

Virtual Machine with pre-installed SyneRBI software
http://www.ccpsynerbi.ac.uk
Apache License 2.0
3 stars 7 forks source link

Python3 build #170

Closed paskino closed 3 years ago

paskino commented 4 years ago

Updates scripts and packages to build with SIRF with python3.

paskino commented 4 years ago

Notice that it is needed to install cython and cython3 because some of the prerequisites in the SuperBuild build with python2 apparently.

paskino commented 4 years ago

This also means that some packages in the SuperBuild pick the wrong python version and I'm not entirely sure which one.

KrisThielemans commented 4 years ago

I think this will be premature to merge for 2.2. We can of course always release another VM afterwards.

Possibly this could be resolved by tackling https://github.com/SyneRBI/SIRF-SuperBuild/issues/48.

rijobro commented 4 years ago

As discussed previously, we could potentially switch to 20.04 meaning that it would use python3 by default. Might fix some problems without any effort on our part.

Would require fixing https://github.com/SyneRBI/SIRF-SuperBuild/issues/413.

KrisThielemans commented 4 years ago

without any effort on our part

seems somewhat wishful thinking, given Alex's struggles :wink:

In any case, finding correct Python versions will remain a bane. Maybe not on a clean Ubuntu 20.04 VM, but as soon as conda gets involved, it'll be a load of fun to be sure we don't mix conda and system python.