Skip to content

ignore deps when installing via pip#557

Merged
inclement merged 1 commit into
kivy:masterfrom
kived:pip-no-deps
Jan 8, 2016
Merged

ignore deps when installing via pip#557
inclement merged 1 commit into
kivy:masterfrom
kived:pip-no-deps

Conversation

@kived

@kived kived commented Jan 8, 2016

Copy link
Copy Markdown
Contributor

This avoids pip trying to overwrite packages which were installed via recipes (and sometimes throwing an error due to the package already existing in the destination, depending on pip version). This does mean you must specify every dependency package.

@inclement

Copy link
Copy Markdown
Member

I was originally worried that having to work out the deps could itself be annoying for users. I'll merge it, but do you think there's a nice way to somehow make it toggleable, or if that's a good idea?

inclement added a commit that referenced this pull request Jan 8, 2016
ignore deps when installing via pip
@inclement inclement merged commit fd93b40 into kivy:master Jan 8, 2016
@kived

kived commented Jan 8, 2016

Copy link
Copy Markdown
Contributor Author

It's difficult, cause pip just kinda sucks. :( No API (well, at least no stable documented API, I'm sure it's possible to just import stuff), you're stuck with the commands they give you. Ubuntu also ships an OLD version of pip which has additional issues with the --target parameter, and this is likely what many people will have. If it would simply look in the right place to determine if a dep is present it would work.

I tried a number of things, including installing pip into hostpython and invoking it via targetpython etc. I even have a install_into_targetpython option in my code to allow for host-only recipes just for this. (I'll probably make a PR with it at some point, but it's unused now. Could be useful if we wanted to install cython in hostpython, which I think is a pretty damn good idea.)

@misl6 misl6 mentioned this pull request Dec 8, 2021
5 tasks
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