Skip to content

Conversation

@aymanhab
Copy link
Member

No description provided.

Test python 3.7 before retiring travis and moving to github Actions
break build into multiline
Use 4.2 to work around package name issues
use conda from incubator on osx
@aymanhab
Copy link
Member Author

aymanhab commented Mar 7, 2022

@moorepants I can use your help getting this over the finish line. In particular testing the windows package and linux package (the latter excluding Moco)
available here
https://anaconda.org/opensim_admin/opensim
and particularly getting the osx package working. Any pointers on setting the right OSX Sdk on github build machines (my local build on osx gets past this without issue). Thanks

@aymanhab
Copy link
Member Author

aymanhab commented Mar 7, 2022

@nickbianco based on this SO trail
https://stackoverflow.com/questions/43145667/anaconda-is-not-creating-full-environment
You should NOT count on the base environment python version, if you want full python you should install it explicitly in the environment you create. This appears to be the case since conda 4.4 please let me know if that works for you.

@nickbianco
Copy link
Member

I always tried to install the version of Python I needed into the specific conda environment I made, but that still didn't work (same result as what Carmichael tried). Based another email you wrote, it seems like everything is fine if the base environment is 3.9 (i.e., 3.7 and 3.9 work on a custom environment). This still isn't great that the base environment needs to be a specific version of Python, but better than the issue we were having before.

@aymanhab
Copy link
Member Author

While this PR builds 4.4 branch and has baked in 4.4 assumptions, will merge into master then cleanup since separating into ongoing dev and 4.4 at this point is worthless as master branch is obsolete.

@aymanhab aymanhab merged commit b1020c3 into master Jun 20, 2022
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.

4 participants