-
Notifications
You must be signed in to change notification settings - Fork 80
Use Rocky Linux 8 for building Python packages #4842
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add readme to opencascade patches?
- name: Install uv | ||
uses: astral-sh/setup-uv@bd01e18f51369d5a26f1651c3cb451d3417e3bba # v6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mb add comment what is it and why is it needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
@@ -56,7 +56,7 @@ def patch_whl(out_dir,libs_dir): | |||
elif SYSTEM == "Linux": | |||
sys.path.append(libs_dir) # to find SO files | |||
# see also: https://github.com/mayeut/pep600_compliance | |||
manylinux_version = "2_31" | |||
manylinux_version = "2_28" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure about that, because we are calling it on ubutnu20
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted.
scripts/wheel/get_sysconfig_var.py
Outdated
#!/usr/bin/env python3 | ||
import sysconfig | ||
|
||
if __name__ == '__main__': | ||
import sys | ||
print(sysconfig.get_config_var(sys.argv[1])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To retrieve system paths for multiple Python instances.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment.
No description provided.