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

[MAN-420] Fix build error concerning mac catalyst sdk version #142

Conversation

ksidirop-laerdal
Copy link
Collaborator

No description provided.

…for the properties <TargetPlatformVersion> and <SupportedOSPlatformVersion> for all platforms

   when building through Laerdal.Builder.targets (which is used by github-actions.yml) these properties are set to the values of the env-vars defined at the top of github-actions.yml via the new msbuild parameters Laerdal_TargetPlatformVersion_IOS/Android/MacCatalyst
…nvoked with --disable-package-restore to avoid some issues with mac-catalyst builds when the mac-catalyst aspect of the build is disabled
…latformVersion> separately from the target sdk that was used to build each binding project

   For iOS and MacCatalyst we use TargetPlatformVersion=17.0 because this is what the macos-14 vm-image of our CI pipelines actually support

   (if we use values less than 17.0 we get a warning which says that in the future this will be considered an error)
…irop/MAN-420-fix-build-error-concerning-mac-catalyst-sdk-version

# Conflicts:
#	.github/workflows/github-actions.yml
#	Laerdal.McuMgr.Bindings.MacCatalyst/Laerdal.McuMgr.Bindings.MacCatalyst.csproj
#	Laerdal.Scripts/Laerdal.Builder.targets
@ksidirop-laerdal ksidirop-laerdal merged commit 01381d2 into develop Oct 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant