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

tests.yml: force vendored Ruby on Catalina #66290

Merged
merged 1 commit into from
Dec 5, 2020
Merged

tests.yml: force vendored Ruby on Catalina #66290

merged 1 commit into from
Dec 5, 2020

Conversation

fxcoudert
Copy link
Member

After their latest update, the Catalina CI nodes are hit by this bug: Homebrew/brew#9410
This is a temporary workaround, so CI testing can resume

@BrewTestBot BrewTestBot added the automerge-skip `brew pr-automerge` will skip this pull request label Dec 5, 2020
@fxcoudert
Copy link
Member Author

Somehow the tests are not running. Adding a formula bump to check what's happening.

@fxcoudert
Copy link
Member Author

Test run shows it works and can build a formula. Rerunning without the test formula, this time.

@fxcoudert fxcoudert merged commit 6fccea4 into Homebrew:master Dec 5, 2020
@fxcoudert fxcoudert deleted the ruby branch December 5, 2020 18:53
@carlocab carlocab mentioned this pull request Dec 5, 2020
Closed
5 tasks
@carlocab
Copy link
Member

carlocab commented Dec 5, 2020 •
edited
Loading

Update: Had a look at a CI run of a different PR and that succeeded on 10.15, so this is probably unrelated. Apologies for the distraction!

From #66287:
10.14 succeeds, but 10.15 build error (after this PR was merged):

==> brew uninstall --force lua [email protected] [email protected]
Error: No available formula with the name "vv".
Error: Process completed with exit code 1.

This was referenced Dec 5, 2020
Closed
Closed
MikeMcQuaid added a commit that referenced this pull request Dec 7, 2020
@MikeMcQuaid
Revert "tests.yml: force vendored Ruby on Catalina (#66290)"
30ce6bc 
This reverts commit 6fccea4.
@MikeMcQuaid MikeMcQuaid mentioned this pull request Dec 7, 2020
Merged
MikeMcQuaid added a commit that referenced this pull request Dec 7, 2020
@MikeMcQuaid
Revert "tests.yml: force vendored Ruby on Catalina (#66290)"
3e46a50 
This reverts commit 6fccea4.
@fxcoudert fxcoudert mentioned this pull request Dec 7, 2020
Merged
sjackman added a commit to sjackman/homebrew-bio that referenced this pull request Dec 7, 2020
@sjackman
CI: Set HOMEBREW_FORCE_VENDOR_RUBY=1 for macOS
a9f441f 
See Homebrew/brew#9442
and Homebrew/brew#9452
and Homebrew/homebrew-core#66290
@sjackman sjackman mentioned this pull request Dec 7, 2020
Merged
4 tasks
sjackman added a commit to brewsci/homebrew-bio that referenced this pull request Dec 7, 2020
@sjackman
CI: Set HOMEBREW_FORCE_VENDOR_RUBY=1 for macOS (#1220)
23d91dd 
See Homebrew/brew#9442
and Homebrew/brew#9452
and Homebrew/homebrew-core#66290
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 5, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MikeMcQuaid MikeMcQuaid Awaiting requested review from MikeMcQuaid MikeMcQuaid is a code owner

Assignees
No one assigned
Labels
automerge-skip `brew pr-automerge` will skip this pull request outdated PR was locked due to age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fxcoudert @carlocab @BrewTestBot