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

Downgrade patchelf version from 0.18.0 to 0.17.2 #295

Merged
merged 4 commits into from
Jan 22, 2025

Conversation

nv-kmcgill53
Copy link
Contributor

Downgrade patchelf version from 0.18.0 to 0.17.2 due to patchelf regression

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

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
tools/Dockerfile.ort Outdated Show resolved Hide resolved
@nv-kmcgill53 nv-kmcgill53 requested a review from rmccorm4 January 22, 2025 19:08
@nv-kmcgill53 nv-kmcgill53 merged commit fd2a4e4 into r25.01 Jan 22, 2025
3 checks passed
mc-nv pushed a commit that referenced this pull request Jan 31, 2025
Downgrade patchelf version from 0.18.0 to 0.17.2 due to patchelf regression

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
mc-nv added a commit that referenced this pull request Feb 4, 2025
* Update CUDA archs in ORT (#291)

* Update CUDA archs in ORT

* Update CUDA archs in ORT and fixing typo

* Update cudnn home

* Version upgrade for openvino updated from 24.12

* Update CUDNN_home for v9.7.0

* Update the home path

* Update path

* Update CUDNN home

* Remove comments

* Update the supported igpu NVCC

* Update the supported igpu and dgpu NVCC

* Upadate default branch post 24.12 (#290)

* Update README and versions for 2.53.0 / 24.12 (#288)

* Update 'gen_ort_dockerfile.py' fil to meet changes in ONNX Runtime 1.20.x

* remove psutils

* Update reformating

* restore description

* Update OpenVINO to 2024.5 (#287)

* Update 'gen_ort_dockerfile.py' fil to meet changes in ONNX Runtime 1.20.x

* remove psutils

* Add OpenVINO version

* Fix pre-commit issues

* Extract archive in different location (#289)

* Extract archive in different location

* Revert "Extract archive in different location"

This reverts commit e57256a.

* Update installation instructions

* Reapply "Extract archive in different location"

This reverts commit 59e24e4.

* Rolling back changes cherry picked from 24.12

* Rolling back changes cherry picked from 24.12: fixing spaces

---------

Co-authored-by: Misha Chornyi <[email protected]>

* Downgrade patchelf version from 0.18.0 to 0.17.2  (#295)

Downgrade patchelf version from 0.18.0 to 0.17.2 due to patchelf regression

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

---------

Co-authored-by: Pavithra Vijayakrishnan <[email protected]>
Co-authored-by: Kyle McGill <[email protected]>
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