Skip to content

Mention build.bat counterpart in GH workflows error message #26770

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 1 commit into from
Jul 27, 2021
Merged

Mention build.bat counterpart in GH workflows error message #26770

merged 1 commit into from
Jul 27, 2021

Conversation

Fidget-Spinner
Copy link
Member

No description provided.

@Fidget-Spinner
Copy link
Member Author

@terryjreedy would you like me to create an issue for this as well (and add news) or do you feel that isn't necessary?

@terryjreedy terryjreedy requested a review from zooba June 17, 2021 16:00
@terryjreedy
Copy link
Member

@zooba Just to make sure, is "build.bat --regen" the proper equivalent of "make regen-all"?

@github-actions
Copy link

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Jul 18, 2021
@ammaraskar
Copy link
Member

@zooba Just to make sure, is "build.bat --regen" the proper equivalent of "make regen-all"?

Not Steve but just wanted to say that it is indeed the correct command, see the PR it was added in for reference: #12654

Adds an additional project to the PCbuild.sln that runs the equivalent commands that are in the regen-all stage in the Makefile.

@ammaraskar ammaraskar removed the stale Stale PR or inactive for long period of time. label Jul 27, 2021
@terryjreedy terryjreedy added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Jul 27, 2021
@terryjreedy
Copy link
Member

Thank you. That confirms -regen is part of 3.9, so can backport this.

@terryjreedy terryjreedy merged commit d61b69f into python:main Jul 27, 2021
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

GH-27382 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jul 27, 2021
@bedevere-bot
Copy link

GH-27383 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 Jul 27, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2021
@Fidget-Spinner @miss-islington
Add windows build.bat counterpart for 'make regen-all' in error messa…
e90bc15 
…ge (pythonGH-26770)

(cherry picked from commit d61b69f)

Co-authored-by: Ken Jin <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 27, 2021
@Fidget-Spinner @miss-islington
Add windows build.bat counterpart for 'make regen-all' in error messa…
404634c 
…ge (pythonGH-26770)

(cherry picked from commit d61b69f)

Co-authored-by: Ken Jin <[email protected]>
miss-islington added a commit that referenced this pull request Jul 27, 2021
@miss-islington @Fidget-Spinner
Add windows build.bat counterpart for 'make regen-all' in error messa…
b631d8d 
…ge (GH-26770)

(cherry picked from commit d61b69f)

Co-authored-by: Ken Jin <[email protected]>
terryjreedy pushed a commit that referenced this pull request Jul 27, 2021
@miss-islington @Fidget-Spinner
Add windows build.bat counterpart for 'make regen-all' in error messa…
a13edfb 
…ge (GH-26770)

(cherry picked from commit d61b69f)

Co-authored-by: Ken Jin <[email protected]>
@terryjreedy
Copy link
Member

Both backports merged.

@Fidget-Spinner Fidget-Spinner deleted the patch-1 branch July 30, 2021 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ammaraskar ammaraskar ammaraskar approved these changes

@zooba zooba Awaiting requested review from zooba

Assignees
No one assigned
Labels
skip issue skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Fidget-Spinner @terryjreedy @ammaraskar @miss-islington @bedevere-bot @the-knights-who-say-ni