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

Build: Update macOS runner (legacy) (macos-10.15) #2773

Closed
ann0see opened this issue Aug 8, 2022 · 11 comments · Fixed by #3066
Closed

Build: Update macOS runner (legacy) (macos-10.15) #2773

ann0see opened this issue Aug 8, 2022 · 11 comments · Fixed by #3066
Assignees
Labels
bug Something isn't working tooling Changes to the automated build system

Comments

@ann0see
Copy link
Member

ann0see commented Aug 8, 2022

Describe the bug

GitHub is deprecating the use of macOS Catalina: actions/runner-images#5583

We'll need to update soon to still support the legacy build.

To Reproduce

/

Expected behavior

/

Screenshots

/

Operating system
macOS

@ann0see ann0see added the bug Something isn't working label Aug 8, 2022
@ann0see ann0see added this to the Release 3.9.1 milestone Aug 8, 2022
@hoffie
Copy link
Member

hoffie commented Aug 12, 2022

We currently use Xcode 12.1.1 for the legacy build. This version is no longer available on macos-11 images.
We either need to downgrade to 11.7 or upgrade to 12.4.

Based on the comments, we can't upgrade to 12.4 as it contains SDK 11.x which is not supported by Qt5:

# For Qt5 on Mac, we need to ensure SDK 10.15 is used, and not SDK 11.x.
# Xcode 12.1 is the most-recent release which still ships SDK 10.15:
# https://developer.apple.com/support/xcode/
# Xcode 12.1.1 is the most-recent 12.1.x release:
# https://xcodereleases.com/

Therefore, it seems like the only valid options would be downgrading to 11.7 even if it sounds totally wrong.

hoffie added a commit to hoffie/jamulus that referenced this issue Aug 12, 2022
Earlier macos versions are deprecated on Github Runners.
Xcode 12.1 is not available on macos-11 images.
Therefore downgrade to 11.7 which is the only version with
SDK 10.15 support on those images

Fixes: jamulussoftware#2773
@ann0see
Copy link
Member Author

ann0see commented Aug 14, 2022

The question is if not supported = produces broken build or just untested.

@hoffie
Copy link
Member

hoffie commented Sep 16, 2022

This issue is almost certainly going to affect us this year. We should somehow figure out how to continue with legacy builds.

  1. Do we now which macOS versions require the legacy version at all?
  2. Are those macOS versions still supported?
  3. Do we know of any users who use such software?

I'd propose to ask at least the latter question in the Discussion section (maybe ping some known Mac users). If they are still on those old releases and have good reasons, we might keep it. Otherwise I'd argue for dropping support for that build.

@ann0see Could you do that?
Pinging @softins here as well in case he has opinions on that... :)

@ann0see
Copy link
Member Author

ann0see commented Sep 17, 2022

@hoffie macOS legacy is being used: https://somsubhra.github.io/github-release-stats/?username=jamulussoftware&repository=jamulus&page=1&per_page=5

Ofc. Not that many people in comparison to the normal Mac version, but still a considerable amount.
I do know quite some (not so tech savvy) musicians who use outdated macs/PCs for music applications like my second choir leader.

Especially buying a new Mac is expensive and a lot of music applications work well on these old machines.

@ann0see
Copy link
Member Author

ann0see commented Sep 17, 2022

We could of course make a policy shift: provide legacy base version, but if we introduce features which don't work out of the box on old Qt versions, we don't provide workarounds (as for softins' language PR, where we'd drop legacy translation support).

@ann0see
Copy link
Member Author

ann0see commented Oct 7, 2022

I think 3.9.1 will be the last version we can build on the old runner.

@ann0see ann0see modified the milestones: Release 3.9.1, Release 3.10.0 Oct 7, 2022
@hoffie hoffie self-assigned this Oct 31, 2022
@hoffie hoffie changed the title Update macOS runner (legacy) Build: Update macOS runner (legacy) (macos-10.15) Oct 31, 2022
hoffie added a commit to hoffie/jamulus that referenced this issue Oct 31, 2022
macos-10.15 will soon stop being available on Github runners.

Fixes: jamulussoftware#2773
@hoffie
Copy link
Member

hoffie commented Oct 31, 2022

Meh, problems here as well: something (brew?) doesn't support xcode 11.7. :(
https://github.com/hoffie/jamulus/actions/runs/3365069827/jobs/5580149907#step:10:1427

Maybe we can try an older brew or switch xcode right before the create-dmg step.

The legacy build is a collection of hacks and workarounds. :(

@hoffie
Copy link
Member

hoffie commented Nov 1, 2022

That's probably the problematic check: https://github.com/Homebrew/brew/blob/master/Library/Homebrew/os/mac/xcode.rb#L363

We could workaround that by switching to a more recent xcode for brew/create-dmg stage. I've tried that here:
https://github.com/hoffie/jamulus/actions/runs/3369510835

Can someone test if the resulting build is usable as a "legacy" build? If so, I'll try to come up with a way to get this into a mergable state...

hoffie added a commit to hoffie/jamulus that referenced this issue Nov 17, 2022
macos-10.15 will soon stop being available on Github runners.

Fixes: jamulussoftware#2773
@hoffie
Copy link
Member

hoffie commented Nov 17, 2022

Previous artifacts expired, I've kicked off a new run here: https://github.com/hoffie/jamulus/actions/runs/3492108643

@pljones
Copy link
Collaborator

pljones commented Apr 19, 2023

@hoffie will this have a PR ready soon?

@pljones pljones added the tooling Changes to the automated build system label Jun 4, 2023
@pljones pljones added this to Tracking Jul 28, 2023
@pljones pljones moved this from Triage to Done in Tracking Jul 28, 2023
@github-project-automation github-project-automation bot moved this to Triage in Tracking Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooling Changes to the automated build system
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants