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.

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

xz 5.2.9 #116778

Closed
wants to merge 1 commit into from
Closed

xz 5.2.9 #116778

wants to merge 1 commit into from

Conversation

p-linnane
Copy link
Member

Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Nov 27, 2022
@p-linnane
Copy link
Member Author

Continuing from #115687

@p-linnane p-linnane added CI-linux-self-hosted Build on Linux self-hosted runner CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. long build Set a long timeout for formula testing CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. labels Nov 27, 2022
@chenrui333 chenrui333 removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Nov 28, 2022
@chenrui333
Copy link
Member

chenrui333 commented Nov 28, 2022 •
edited
Loading

pyflow failure seems legit, raise an issue in the upstream
I could not reproduce s3cmd and waybackpy issues, but I am using 13-arm machine

The linux build failures seem irrelevant to this PR

@github-actions

This comment was marked as outdated.

Sign in to view
@github-actions github-actions bot added the stale No recent activity label Nov 30, 2022
@p-linnane p-linnane added in progress Stale bot should stay away and removed stale No recent activity labels Nov 30, 2022
@p-linnane p-linnane changed the title xz 5.2.8 xz 5.2.9 Nov 30, 2022
@p-linnane p-linnane added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Nov 30, 2022
@p-linnane
Copy link
Member Author

Rebased with 5.2.9 and re-running.

@cho-m
Copy link
Member

cho-m commented Dec 1, 2022

pyflow failure occurs in other PRs. It looks like it queries dependencies via https://github.com/David-OConnor/pydeps but the API (e.g. https://pydeps.herokuapp.com/requests) doesn't seem to work right now.

@p-linnane p-linnane removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 2, 2022
@p-linnane
Copy link
Member Author

@cho-m Is pyflow our blocker here? Seems like the pydeps API has been down for some time.

@fxcoudert
Copy link
Member

Given the huge number of dependencies, and the nature of the failures seen, I think this can be merged. @cho-m any objection?

@fxcoudert fxcoudert added ready to merge PR can be merged once CI is green and removed in progress Stale bot should stay away labels Dec 4, 2022
cho-m
cho-m approved these changes Dec 4, 2022
View reviewed changes
Copy link
Member

@cho-m cho-m left a comment

Choose a reason for hiding this comment

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

Should be fine to merge. Need to manually create archive.org mirrors though, particularly the HTTP one.

@p-linnane
Copy link
Member Author

p-linnane commented Dec 4, 2022 •
edited
Loading

Do we have any documentation on that process anywhere?

Nevermind. Sorted it out and it's live now.

@BrewTestBot
Copy link
Member

:shipit: @p-linnane has triggered a merge.

@p-linnane p-linnane deleted the bump-xz-5.2.8 branch December 4, 2022 18:38
@github-actions github-actions bot added the outdated PR was locked due to age label Jan 4, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fxcoudert fxcoudert fxcoudert approved these changes

@cho-m cho-m cho-m approved these changes

Assignees
No one assigned
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-linux-self-hosted Build on Linux self-hosted runner CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. long build Set a long timeout for formula testing outdated PR was locked due to age ready to merge PR can be merged once CI is green
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@p-linnane @chenrui333 @cho-m @fxcoudert @BrewTestBot