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

Mac: Upgrade SDK and macOS version for legacy build #3066

Merged
merged 1 commit into from
May 19, 2023

Conversation

ann0see
Copy link
Member

@ann0see ann0see commented May 11, 2023

Short description of changes
Updates the SDK to an unsupported higher version for the legacy build. I think, if it builds, it builds and could maybe work. Still, we should slowly deprecate the legacy build (but not suddenly)
I've successfully built it on my repo: https://github.com/ann0see/jamulus/actions/runs/4952341059

Upgrades the legacy build to use the last supported SDK.

CHANGELOG: Mac: The legacy build now uses the macOS 13.1 SDK which is unsupported by Qt 5.9.9. We still provide the legacy build, but from now on consider it as deprecated. You are advised to upgrade your OS.

Context: Fixes an issue?
Fixes: #2773

Does this change need documentation? What needs to be documented and how?

Status of this Pull Request
Needs testing by someone with access to an old OS. Or at least someone with macOS. @gilgongo ?

What is missing until this pull request can be merged?

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see added this to the Release 3.10.0 milestone May 11, 2023
@ann0see ann0see added this to Triage in Tracking (old) via automation May 11, 2023
@ann0see ann0see moved this from Triage to Waiting on Team in Tracking (old) May 11, 2023
@gilgongo
Copy link
Member

I'm seeing the active tab being blank here:

image

@ann0see
Copy link
Member Author

ann0see commented May 16, 2023

I think it's been like that for a while on modern OS.

@ann0see
Copy link
Member Author

ann0see commented May 17, 2023

I think it's ok to get in then. (Legacy should slowly get deprecated)

Copy link
Member

@gilgongo gilgongo left a comment

Choose a reason for hiding this comment

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

Can't see anything else that might be a problem.

@ann0see ann0see requested a review from pljones May 18, 2023 06:01
@ann0see
Copy link
Member Author

ann0see commented May 18, 2023

@pljones please merge if you're ok with that. I believe that it's ok - but I can't really test it.

Copy link
Collaborator

@pljones pljones left a comment

Choose a reason for hiding this comment

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

I know nothing at all about macOS.

Tracking (old) automation moved this from Waiting on Team to In Progress May 19, 2023
@pljones pljones merged commit 068bfbc into jamulussoftware:main May 19, 2023
10 of 11 checks passed
Tracking (old) automation moved this from In Progress to Done May 19, 2023
@ann0see ann0see deleted the autobuild/fixLegacy branch May 19, 2023 21:14
@pljones pljones added the tooling Changes to the automated build system label Jun 4, 2023
@pljones pljones removed this from Done in Tracking (old) Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Changes to the automated build system
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Build: Update macOS runner (legacy) (macos-10.15)
3 participants