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

Use build-in modification of pip.conf #29

Merged
merged 5 commits into from
Aug 2, 2024
Merged

Use build-in modification of pip.conf #29

merged 5 commits into from
Aug 2, 2024

Conversation

TillBeemelmanns
Copy link
Member

@TillBeemelmanns TillBeemelmanns commented Aug 1, 2024

  • Uses pip config command to edit /root/.config/pip/pip.conf
  • https://pip.pypa.io/en/latest/cli/pip_config/
  • No more collisions possible
  • Previous behavior: Possibly two sections [global] in /root/.config/pip/pip.conf would have been created which causes pip to crash

Note that

# [Priority 1] Site level configuration files
#       1. `/usr/pip.conf`
#
# [Priority 2] User level configuration files
#       1. `/root/.config/pip/pip.conf`
#       2. `/root/.pip/pip.conf`
#
# [Priority 3] Global level configuration files
#       1. `/etc/pip.conf`
#       2. `/etc/xdg/pip/pip.conf`

Hence we need to execute:

# for /etc/pip.conf
RUN python -m pip config --global set global.break-system-packages true

@TillBeemelmanns TillBeemelmanns changed the title Added safe modification of pip.conf Add safe modification of pip.conf Aug 1, 2024
@TillBeemelmanns TillBeemelmanns changed the title Add safe modification of pip.conf Use build-in modification of pip.conf Aug 1, 2024
@jpbusch jpbusch merged commit 8f10aa1 into ika-rwth-aachen:main Aug 2, 2024
2 of 6 checks passed
@TillBeemelmanns TillBeemelmanns deleted the fix/safe_pip_conf_modification branch August 2, 2024 13:55
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