Skip to content

Fix building native python dependencies for pip-compile#2815

Merged
feelepxyz merged 1 commit intomainfrom
jurre/fix-pip-compile-native-dependencies
Dec 8, 2020
Merged

Fix building native python dependencies for pip-compile#2815
feelepxyz merged 1 commit intomainfrom
jurre/fix-pip-compile-native-dependencies

Conversation

@jurre
Copy link
Copy Markdown
Member

@jurre jurre commented Dec 2, 2020

Some native python dependencies require to be built from scratch,
because no Wheel is available for them for the given version of Python.

If this is the case, when trying to install with the --build-isolation
flag, pip requires all the dependencies are available as wheels, but
this is often not the case.

Building with the --no-build-isolation ensures that any transitive
dependencies are also installed/compiled from source.

@jurre jurre force-pushed the jurre/fix-pip-compile-native-dependencies branch from aff4979 to 05af702 Compare December 3, 2020 14:41
Some native python dependencies require to be built from scratch,
because no Wheel is available for them for the given version of Python.

If this is the case, when trying to install with the `--build-isolation`
flag, pip requires all the dependencies are available as wheels, but
this is often not the case.

Building with the `--no-build-isolation` ensures that any transitive
dependencies are also installed/compiled from source.
@jurre jurre force-pushed the jurre/fix-pip-compile-native-dependencies branch from 05af702 to 21ed214 Compare December 7, 2020 16:51
@jurre jurre changed the title [WIP] Fix building native python dependencies for pip-compile Fix building native python dependencies for pip-compile Dec 7, 2020
@jurre jurre marked this pull request as ready for review December 7, 2020 17:56
@jurre jurre requested a review from a team as a code owner December 7, 2020 17:56
@jurre jurre requested a review from feelepxyz December 7, 2020 17:56
@jurre
Copy link
Copy Markdown
Member Author

jurre commented Dec 7, 2020

We'll probably want to do something similar for regular pip builds, but no reason we should hold off merging the fix for pip compile IMO.

Copy link
Copy Markdown
Contributor

@feelepxyz feelepxyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@feelepxyz feelepxyz merged commit f324db2 into main Dec 8, 2020
@feelepxyz feelepxyz deleted the jurre/fix-pip-compile-native-dependencies branch December 8, 2020 09:35
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