search

CVMix / CVMix-src

CVMix source code (including protex documentation) as well as build system and examples / tests
Other
30 stars 29 forks source link

Update makedep.py to support python 3 #76

Closed xylar closed 5 years ago

xylar commented 5 years ago

Currently, CVMIX can only be built (at least via MPAS-Ocean) with python 2.7. Given that python 2 is going away later this year, it seems necessary to support python 3 as well.

With this merge, the python environment used to build CVMIx will need the future package in python 2.7. If this dependency is a problem, it could be removed with the side effect that print output will look strange in python 2.7 (be surrounded by parentheses).

xylar commented 5 years ago

Testing

I have been able to successfully build MPAS-Ocean including CVMix with these changes in both a python 2.7 and python 3.7 environment. A build with python 3.7 failed without these changes (specifically the print functions).

mnlevy1981 commented 5 years ago

Thanks so much for doing this!

xylar commented 5 years ago

Awesome! Thanks for the quick turn-around on this, @mnlevy1981.