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.

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

[SDK-3926] Backport #1823 to fix/socketrocket-fix #1825

Closed

Conversation

lawrence-forooghian
Copy link
Collaborator

@lawrence-forooghian lawrence-forooghian commented Nov 6, 2023

We have a customer who is using the branch fix/socketrocket-fix, which contains the commit c2ed307 that’s not on main. Although we recognise that we need to transition them off that branch and onto our latest releases, we've decided we'd like to offer them a version of the fix for #1817, from PR #1823, on their branch.

This PR gets CI working on that branch again and then cherry-picks the fix from #1823 into the branch.

@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 17:50 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from b9b43ec to f6f3875 Compare November 6, 2023 18:10
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 18:17 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from f6f3875 to c943fec Compare November 6, 2023 18:39
`macos-10.15`, which worked whenever it was that we branched to
`fix/socketrocket-fix`, is no longer available. So, let’s explicitly use
macOS 11, which would have been what `macos-latest` pointed to at the
time of c2ed307.
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 18:48 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from c943fec to 82a192c Compare November 6, 2023 18:49
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 18:55 Inactive
The installation now fails on the macos-11 runner:

> Warning: You are using macOS 11.
> We (and Apple) do not provide support for this old version.
> It is expected behaviour that some formulae will fail to build in this old version.
> It is expected behaviour that Homebrew will be buggy and slow.
> Do not create any issues about this on Homebrew's GitHub repositories.
> Do not create any issues even if you think this message is unrelated.
> Any opened issues will be immediately closed without response.
> Do not ask for help from Homebrew or its maintainers on social media.
> You may ask for help in Homebrew's discussions but are unlikely to receive a response.
> Try to figure out the problem yourself and submit a fix as a pull request.
> We will review it but may or may not accept it.
>
> xcbeautify: A full installation of Xcode.app 14.0 is required to compile
> this software. Installing just the Command Line Tools is not sufficient.
>
> Xcode 14.0 cannot be installed on macOS 11.
> You must upgrade your version of macOS.
> Error: xcbeautify: An unsatisfied requirement failed this build.
> Error: Process completed with exit code 1.
This is the earliest version of iOS and tvOS now available in Xcode
13.2.1 (the version of Xcode that would have been the default on the
macos-latest runner at the time of c2ed307).
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from 82a192c to 3b0b1e9 Compare November 6, 2023 19:00
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 19:06 Inactive
Copied from commit 46ee0b0, with a change to `if let foo {` syntax to
get it compiling in older Xcode. Needed for cherry-picking the test
changes from that commit.
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from 3b0b1e9 to c895341 Compare November 6, 2023 19:10
@lawrence-forooghian lawrence-forooghian changed the title WIP backport #1823 to fix/socketrocket-fix [SDK-3926] WIP: Backport #1823 to fix/socketrocket-fix Nov 6, 2023
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 19:16 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 19:41 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 19:53 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 20:03 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 20:19 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from 149a3f2 to 8626008 Compare November 6, 2023 20:20
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 20:27 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from 8626008 to f86a7e2 Compare November 6, 2023 20:29
@github-actions github-actions bot temporarily deployed to staging/pull/1825/jazzydoc November 6, 2023 20:35 Inactive
This is a cherry-pick of 86b7cc7, to resolve #1823 on the
fix/socketrocket-fix branch which one of our customers is using.
@lawrence-forooghian lawrence-forooghian force-pushed the socketrocket-fix-plus-PR-1823 branch from f86a7e2 to 3122fd4 Compare November 7, 2023 12:38
@lawrence-forooghian lawrence-forooghian changed the title [SDK-3926] WIP: Backport #1823 to fix/socketrocket-fix [SDK-3926] Backport #1823 to fix/socketrocket-fix Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant