Skip to content

Conversation

@hoodmane
Copy link
Contributor

@hoodmanehoodmane commented Nov 12, 2024

Pulls in the updated docker container from this PR: python/cpython-devcontainers#34 which contains the autoconf-archive PR:
autoconf-archive/autoconf-archive#316

I removed the workaround from configure.ac, regenerated everything, and manually tested that the fix works.

@erlend-aaslanderlend-aasland changed the title gh-89640 Pull in update to float word order detection in autoconf-archivegh-89640: Pull in update to float word order detection in autoconf-archiveNov 12, 2024
@erlend-aasland
Copy link
Contributor

I guess the check-if-autoconf-files-are-up-to-date CI also needs an update; see .github/workflows/build.yml IIRC.

…nf-archive Pulls in the updated docker container from this PR: python/cpython-devcontainers#34 which contains the autoconf-archive PR: autoconf-archive/autoconf-archive#316 I removed the workaround from configure.ac, regenerated everything, and manually tested that the fix works.
@hoodmane
Copy link
ContributorAuthor

Okay, autoconf up to date check passes now.

Copy link
Contributor

@erlend-aaslanderlend-aasland left a comment

Choose a reason for hiding this comment

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

Thanks!

For future reference: we normally try to avoid force-pushing, since it can make reviewing harder.

@erlend-aaslanderlend-aasland merged commit 142104c into python:mainNov 13, 2024
picnixz pushed a commit to picnixz/cpython that referenced this pull request Dec 8, 2024
@hoodmanehoodmane deleted the update-float-word-detection branch December 9, 2024 15:26
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@hoodmane@erlend-aasland