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

MDA_USE_OPENMP=FALSE doesn't deactivate openMP #629

Closed
kain88-de opened this issue Jan 17, 2016 · 0 comments
Closed

MDA_USE_OPENMP=FALSE doesn't deactivate openMP #629

kain88-de opened this issue Jan 17, 2016 · 0 comments

Comments

@kain88-de
Copy link
Member

The new flags introduced in #471 cannot be used to deactivate openMP. The problem is that the value is parsed as a string and no type conversion takes place.

@orbeckst orbeckst added this to the 0.13 bugfixes milestone Jan 17, 2016
@richardjgowers richardjgowers removed this from the 0.14 milestone Jan 20, 2016
mnmelo added a commit that referenced this issue Feb 9, 2016
Default behavior now depends on release or dev status. When in dev mode
default is to use Cython and delete cythonized files. When in release
mode default is to skip Cython but keep existing cythonized files.

setup.py now reads the version string directly from version.py.

Environment variable handling in setup.py enhanced to recognize strings
with boolean meaning (fixes #629).

Travis' minimal build now also rebuilds cythonized files.
richardjgowers added a commit that referenced this issue Feb 9, 2016
Cythonized files now deleted after setup. (closes #667 / #629)
@richardjgowers richardjgowers added this to the 0.14 milestone Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants