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

gh-98948: Remove obsolete readelf dependency #98949

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

serge-sans-paille commented Nov 1, 2022

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.

@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 Nov 2, 2022
@serge-sans-paille
Copy link
Contributor Author

serge-sans-paille commented Nov 2, 2022

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

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.
@vstinner vstinner merged commit 8c4de57 into python:main Nov 3, 2022
15 checks passed
@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.

None yet

3 participants