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

Upadate default branch post 25.01 #110

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mc-nv
Copy link
Contributor

@mc-nv mc-nv commented Jan 31, 2025

Patchelf shipped a regression in 0.18.0 and has since yanked the pypi release pointing to
0.17.2 as the most recent version. However, 0.18.0 is still the version shipped in both the apt and yum
repositories, thus we must use pip to install the version we want.
See mayeut/patchelf-pypi#87

…ession (#109)

Patchelf shipped a regression in 0.18.0 and has since yanked the pypi release pointing to
0.17.2 as the most recent version. However, 0.18.0 is still the version shipped in both the apt and yum
repositories, thus we must use pip to install the version we want.
See mayeut/patchelf-pypi#87
@@ -187,7 +187,8 @@ explicitly provided, the default value for this option is 4.
Use a recent cmake to build. First install the required dependencies.

```
$ apt-get install patchelf rapidjson-dev
$ apt-get install rapidjson-dev python3-pip

Choose a reason for hiding this comment

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

Why can't we pin the installation with apt itself? So L190 would look like:
apt-get install patchelf==0.17.2 rapidjson-dev

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

Successfully merging this pull request may close these issues.

3 participants