Skip to content

Fix installation script - Restrict pip version to below 23 during setuptools upgrade#31

Open
lucas-ebi wants to merge 1 commit intodevelopfrom
fix-install
Open

Fix installation script - Restrict pip version to below 23 during setuptools upgrade#31
lucas-ebi wants to merge 1 commit intodevelopfrom
fix-install

Conversation

@lucas-ebi
Copy link

@lucas-ebi lucas-ebi commented Nov 15, 2025

Fix pip version compatibility in OneDep installation script

Problem

The OneDep installation script was failing during the setuptools upgrade phase due to compatibility issues with newer versions of pip (23+). These newer versions removed the --global-option flag that is required by some legacy package installations in the OneDep build process.

Solution

This PR restricts pip to versions below 23 during the setuptools upgrade step to maintain compatibility with the existing OneDep build infrastructure.

Changes

  • Modified install_onedep.sh to pin pip version to <23 during the virtual environment setup
  • Added version constraint: pip install --no-cache-dir --upgrade setuptools "pip<23"

Testing

  • Verified OneDep installation completes successfully with the version constraint
  • Confirmed no breaking changes to existing functionality

Impact

This is a minimal, targeted fix that ensures installation stability while maintaining backward compatibility. The version constraint only affects the installation process and does not impact runtime functionality.

Pin pip to versions below 23 to maintain compatibility with older setuptools
installation methods. Newer pip versions (23+) removed the --global-option
flag which is required by some legacy package installations in the OneDep
build process.
@lucas-ebi
Copy link
Author

@epeisach looks like master is ahead of develop, please have a look.

@epeisach
Copy link
Contributor

This is not sustainable. We need PKG_CONFIG_DIR set before installation. I came across this recently as well...

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