Skip to content

bpo-40432: Use python 3.8 or higher to compile cpython on windows #25389

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Apr 26, 2021
Merged

bpo-40432: Use python 3.8 or higher to compile cpython on windows #25389

merged 5 commits into from
Apr 26, 2021

Conversation

Fidget-Spinner
Copy link
Member

@Fidget-Spinner Fidget-Spinner commented Apr 13, 2021 •
edited
Loading

Some regen grammar scripts use the walrus := operator and so they need Python 3.8 or higher to compile. Hopefully this fixes the AMD64 Windows10 Pro build.

Co-Authored-By: Jeremy Kloth 20042737+jkloth@users.noreply.github.com

https://bugs.python.org/issue40432

@Fidget-Spinner Fidget-Spinner mentioned this pull request Apr 13, 2021
Merged
@Fidget-Spinner Fidget-Spinner changed the title Use python 3.8 or higher to compile cpython on windows bpo-40432: Use python 3.8 or higher to compile cpython on windows Apr 13, 2021
@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 13, 2021
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @pablogsal for commit 781fc83 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 13, 2021
@jkloth
Copy link
Contributor

jkloth commented Apr 13, 2021

The PR does indeed solve the issue. The worker compiles and completes the tests (with warnings, but that is unrelated to this change)

@Fidget-Spinner Fidget-Spinner marked this pull request as ready for review April 13, 2021 16:04
@Fidget-Spinner Fidget-Spinner requested a review from a team as a code owner April 13, 2021 16:04
jkloth
jkloth approved these changes Apr 13, 2021
View reviewed changes
Copy link
Contributor

@jkloth jkloth left a comment

Choose a reason for hiding this comment

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

LGTM and passes the PR check!

@Fidget-Spinner
Copy link
Member Author

The PR does indeed solve the issue. The worker compiles and completes the tests (with warnings, but that is unrelated to this change)

Thanks for taking the time to set up a buildbot worker! Without it this may have gone unnoticed longer.

@pablogsal pablogsal requested a review from zooba April 13, 2021 23:36
@pablogsal
Copy link
Member

@Fidget-Spinner please, ping me if @zooba didn't have time to review this in the following days so I can review it.

@Fidget-Spinner @jkloth
Update find_python.bat
83689f0 
Apply suggestions by Jeremy

Co-Authored-By: Jeremy Kloth <20042737+jkloth@users.noreply.github.com>
@Fidget-Spinner Fidget-Spinner added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 14, 2021
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @Fidget-Spinner for commit 83689f0 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 14, 2021
@jkloth
Copy link
Contributor

jkloth commented Apr 17, 2021

While this is still a valid issue with the build scripts, I have now installed 3.8 on my buildbot so as to finally see it useful.

The reason my buildbot has this problem is that I have installed Visual Studio 2019 with the Python component selected. This component also installs Py3.7. David's buildbot uses the 2017 Build Tools which means the logic in find_python falls through to fetching the latest build from nuget.

zooba
zooba reviewed Apr 23, 2021
View reviewed changes
@zooba zooba merged commit 425434d into python:master Apr 26, 2021
@bedevere-bot
Copy link

@zooba: Please replace # with GH- in the commit message next time. Thanks!

@zooba
Copy link
Member

zooba commented Apr 26, 2021

That test failure was asyncio, unrelated to this change.

@zooba zooba added the needs backport to 3.9 only security fixes label Apr 26, 2021
@miss-islington
Copy link
Contributor

Thanks @Fidget-Spinner for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot
Copy link

GH-25637 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Apr 26, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 26, 2021
@Fidget-Spinner @miss-islington
bpo-40432: Use python 3.8 or higher to compile CPython on Windows (py…
ed18b5d 
…thonGH-25389)

(cherry picked from commit 425434d)

Co-authored-by: Ken Jin <28750310+Fidget-Spinner@users.noreply.github.com>
ambv pushed a commit that referenced this pull request Apr 26, 2021
@miss-islington @Fidget-Spinner
bpo-40432: Use python 3.8 or higher to compile CPython on Windows (GH…
f65f3f0 
…-25389) (#25637)

(cherry picked from commit 425434d)

Co-authored-by: Ken Jin <28750310+Fidget-Spinner@users.noreply.github.com>
@Fidget-Spinner
Copy link
Member Author

@zooba , thanks for reviewing this PR and merging it!

@Fidget-Spinner Fidget-Spinner deleted the find-python-3.8 branch April 28, 2021 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zooba zooba zooba left review comments

@jkloth jkloth jkloth approved these changes

@pablogsal pablogsal Awaiting requested review from pablogsal

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Fidget-Spinner @bedevere-bot @jkloth @pablogsal @zooba @miss-islington @the-knights-who-say-ni