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

MacOS and windows pre-compiles binaries #555

Closed
samuelcolvin opened this issue May 26, 2019 · 3 comments
Closed

MacOS and windows pre-compiles binaries #555

samuelcolvin opened this issue May 26, 2019 · 3 comments
Labels
feature request help wanted Pull Request welcome

Comments

@samuelcolvin
Copy link
Member

samuelcolvin commented May 26, 2019

pydantic will soon be compiled with cython, however pre-compiled binaries are only available on linux at the moment.

It should be possible for macos on travis and windows on azure pipeline.

However since I only use linux, it's not my primary concern.

I'd be happy to accept a PR if someone is prepared to implement one or both.

@anentropic
Copy link
Contributor

Just a note, the current "cythonize" step which happens in setup.py gave me the cythonized version of pydantic by default, via wheel, when I pip install pydantic on macos.

This is all great, but I noticed because I was trying to debug something and the cythonized code does not play well with ipdb - you get a stack trace ok but any values you try to inspect give NameError.

To get a plain python version of pydantic I could do this:

pip uninstall pydantic
export SKIP_CYTHON=1
pip install --no-cache-dir --no-binary :all: pydantic

If things change in future it'd be nice to still have a way to install non-cythonized pydantic if necessary.

To be clear, I'm quite happy getting the cythonized version by default though.

@samuelcolvin
Copy link
Member Author

thanks for pointing out this should have already been closed.

You can install non compiled pydantic with

pip install --no-binary pydantic pydantic

I think you'll also need to either not have cython installed or use SKIP_CYTHON=1.

@samuelcolvin
Copy link
Member Author

macos binaries have been being built for some time, binaries are also built for 64 bit windows.

I'll correct the message in the docs. now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request help wanted Pull Request welcome
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants