Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Fix Makefile for Debian Stretch (stable) #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

haimgel
Copy link

@haimgel haimgel commented Jul 3, 2017

Debian 9 introduced patch into pip, which auto-adds "--user" parameter
to pip, which cannot work with "--target". Therefore, "make" fails on
current Debian stable (Stretch, 9). Just adding "--system" before "-t"
fixes that. Yes, it's stupid, but it works. For more info, see:
pypa/pip#3826

to pip, which cannot work with "--target". Therefore, "make" fails on
current Debian stable (Stretch, 9). Just adding "--system" before "-t"
fixes that. Yes, it's stupid, but it works. For more info, see:
pypa/pip#3826
@auchter
Copy link
Owner

auchter commented Jul 20, 2017

While this might fix the build for Debian 9's patched pip, it looks like this will break with upstream pip.

If adding --system was made to be dependent upon the contents / existence of /etc/debian_version, that might be more acceptable...

@haimgel
Copy link
Author

haimgel commented Jul 24, 2017

Yeah, I wasn't aware that Debian added the whole "--system" option, I thought they just changed the default. In any case, I checked this commit on CentOS and Debian, and it seems to be ok. I think checking whether "--system" option actually exists is less brittle than checking for the specific OS version.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants