Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-98948: Remove obsolete readelf dependency #98949

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

Merged
merged 1 commit into from
Nov 3, 2022

Conversation

serge-sans-paille
Copy link
Contributor

@serge-sans-paille serge-sans-paille commented Nov 1, 2022

This got introduced in commit 5884449 to determine if readline is already linked against curses or tinfo in the setup.py, which is no longer present.

@serge-sans-paille
Copy link
Contributor Author

cc @vstinner , because I know he's in love with autoconf. I didn't run a local autoreconf because my local version is different from the one used to generate previous version. Instead, I manually edited the configure script, feel free to regenerate it on your side.

@serge-sans-paille serge-sans-paille force-pushed the remove/readelf-dependency branch from 26b6505 to 033f5f0 Compare November 1, 2022 07:16
@vstinner
Copy link
Member

vstinner commented Nov 2, 2022

Can you try to add a NEWS entry in the Build category to document your change? See https://devguide.python.org/ about blurb.

@vstinner
Copy link
Member

vstinner commented Nov 2, 2022

I added a commit to your branch: I ran "make regen-configure" which uses a container to get always the same version.

@serge-sans-paille serge-sans-paille force-pushed the remove/readelf-dependency branch 2 times, most recently from e425077 to 0fae64f Compare November 2, 2022 10:07
@serge-sans-paille
Copy link
Contributor Author

@vstinner : entry added & commits squashed. Thanks for the review!

@serge-sans-paille serge-sans-paille force-pushed the remove/readelf-dependency branch from 0fae64f to d69d8ad Compare November 3, 2022 12:57
This got introduced in commit 5884449
to determine if readline is already linked against curses or tinfo in
the setup.py, which is no longer present.
@serge-sans-paille serge-sans-paille force-pushed the remove/readelf-dependency branch from d69d8ad to 0de323b Compare November 3, 2022 12:58
@vstinner vstinner merged commit 8c4de57 into python:main Nov 3, 2022
@vstinner
Copy link
Member

vstinner commented Nov 3, 2022

Merged, thanks!

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.

3 participants