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

disable build isolation #6

Closed
wants to merge 1 commit into from
Closed

Conversation

jjhelmus
Copy link

Build isolation is enabled by the presence of the pyproject.toml file in the
dask-ml project. On non-x86 platforms, Cython is not provided as a binary
distributions (wheel file) which causes the build to fail. Disabling build
isolation allows dask-ml to be built on these platforms using pip 10.

Another option to to build using a "python setup.py install ..." command
on these platforms.

Build isolation is enabled by the presence of the pyproject.toml file in the
dask-ml project. On non-x86 platforms, Cython is not provided as a binary
distributions (wheel file) which causes the build to fail. Disabling build
isolation allows dask-ml to be built on these platforms using pip 10.

Another option to to build using a "python setup.py install ..." command
on these platforms.
@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jjhelmus
Copy link
Author

This might fail since pip 9 is used in conda-forge but the argument is needed when building on non-x86 platforms using pip 10.

@jjhelmus
Copy link
Author

Ick, --no-build-isolation is not a valid argument in pip 9, but is needed in pip 10. I have to think about a better solution to this.

@jjhelmus jjhelmus closed this Jun 19, 2018
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.

2 participants