-
-
Notifications
You must be signed in to change notification settings - Fork 776
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
build-recipes: Add --keep-going option #4491
base: main
Are you sure you want to change the base?
Conversation
This adds a --keep-going option to build-recipes that will continue building packages even if some builds fail. As a followup, we should figure out how to stick the failed list somewhere we can easily use to: - revert updates to packages that failed to build - disable packages that failed to build - pin packages that failed to the current version so that auto update ignores them
for more information, see https://pre-commit.ci
color = SUCCESS_BLUE if success else FAIL_RED | ||
print(color + done_message + END_COLOR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using logger.error
here just leaves an empty line on the console. I couldn't figure out how to get it to print correctly. I think this is a reasonable alternative.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's very strange. Could you add a comment about it so we can revisit this later?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Indeed this will be useful when we upgrade packages.
@@ -167,6 +167,10 @@ def build_recipes( | |||
no_deps: bool = typer.Option( | |||
False, help="Removed, use `pyodide build-recipes-no-deps` instead." | |||
), | |||
keep_going: bool = typer.Option( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this command will not be very useful except for maintainers. So maybe add hidden=True
to hide if from the help text?
) | ||
logger.error( | ||
"Failed to build the following packages: " | ||
f"[bold]{format_name_list(build_state.failed_list)}[/bold]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not sort failed and skipped list too?
color = SUCCESS_BLUE if success else FAIL_RED | ||
print(color + done_message + END_COLOR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's very strange. Could you add a comment about it so we can revisit this later?
This adds a --keep-going option to build-recipes that will continue building packages even if some builds fail.
I think this will make updating the Python version or a bunch of packages less annoying.
As a followup, we should figure out how to stick the failed list somewhere we can easily use to:
Checklist