Skip to content

Commit

Permalink
fix (github-actions.yml): upgrade to dotnet workload 8.0.405 and ios/…
Browse files Browse the repository at this point in the history
…mac-catalyst sdks 18.0
  • Loading branch information
ksidirop-laerdal committed Jan 16, 2025
1 parent 4f602ef commit 15b67a9
Show file tree
Hide file tree
Showing 6 changed files with 35 additions and 33 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/github-actions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,17 +20,17 @@ env:
SCL_DEPENDENCY_TRACKER_SERVER_URL: ${{ secrets.SCL_DEPENDENCY_TRACKER_SERVER_URL }}
SCL_DEPENDENCY_TRACKER_SIGNING_PRIVATE_KEY: ${{ secrets.SCL_DEPENDENCY_TRACKER_SIGNING_PRIVATE_KEY }}

DOTNET_TARGET_WORKLOAD_VERSION: "8.0.402" # dont upgrade this lightheartedly the workload snapshot implicitly defines which versions of Android/iOS/MacCatalyst SDKs are supported
DOTNET_TARGET_WORKLOAD_VERSION: "8.0.405" # dont upgrade this lightheartedly the workload snapshot implicitly defines which versions of Android/iOS/MacCatalyst SDKs are supported

BINDINGS_ANDROID___DOTNET_TARGET_PLATFORM_VERSION: "34" # for the csproj file

BINDINGS_IOS___SDK_VERSION: "18.1" # for xcodebuild
BINDINGS_IOS___XCODE_IDE_DEV_PATH: "/Applications/Xcode_16.1.app/Contents/Developer" # for xcodebuild
BINDINGS_IOS___DOTNET_TARGET_PLATFORM_VERSION: "17.0" # for the csproj file
BINDINGS_IOS___DOTNET_TARGET_PLATFORM_VERSION: "18.0" # for the csproj file

BINDINGS_MACCATALYST___SDK_VERSION: "15.1" # for xcodebuild
BINDINGS_MACCATALYST___XCODE_IDE_DEV_PATH: "/Applications/Xcode_16.1.app/Contents/Developer" # for xcodebuild
BINDINGS_MACCATALYST___DOTNET_TARGET_PLATFORM_VERSION: "17.0" # for the csproj file
BINDINGS_MACCATALYST___DOTNET_TARGET_PLATFORM_VERSION: "18.0" # for the csproj file

on:
workflow_call: # so that other workflows can trigger this
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,10 @@
<GeneratePackageOnBuild>true</GeneratePackageOnBuild>

<!-- these versions are getting replaced by the build script in one swift pass -->
<Version>1.0.1092.0</Version>
<FileVersion>1.0.1092.0</FileVersion>
<PackageVersion>1.0.1092.0</PackageVersion>
<AssemblyVersion>1.0.1092.0</AssemblyVersion>
<Version>1.0.1208.0</Version>
<FileVersion>1.0.1208.0</FileVersion>
<PackageVersion>1.0.1208.0</PackageVersion>
<AssemblyVersion>1.0.1208.0</AssemblyVersion>

<Title>$(PackageId)</Title>
<Owners>$(Authors)</Owners>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

<!-- this should must be kept in sync between this file and laerdal.mcumgr.csproj notice that the ordering of the following evaluations is important! -->
<TargetPlatformVersion Condition=" '$(IsNetXMacCatalyst)' == 'true' and '$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)' != '' ">$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsNetXMacCatalyst)' == 'true' and '$(TargetPlatformVersion)' == '' ">17.0</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsNetXMacCatalyst)' == 'true' and '$(TargetPlatformVersion)' == '' ">18.0</TargetPlatformVersion>
<SupportedOSPlatformVersion Condition=" '$(IsNetXMacCatalyst)' == 'true' ">13.1</SupportedOSPlatformVersion>

<!-- setting IsBindingProject to true is vital in order for the btouch tools to actually -->
Expand All @@ -43,7 +43,7 @@
<NativeFrameworkActualTargetFolderpath>$(NativeFrameworkParentFolderpath)/McuMgrBindingsiOS.framework</NativeFrameworkActualTargetFolderpath>

<!-- set these explicitly here only for experimentation as far as cicd is concerned these values should be set in the .yml file of the pipeline -->
<!-- <Laerdal_Bindings_MacCatalyst___Sdk_Version Condition=" '$(Laerdal_Bindings_MacCatalyst___Sdk_Version)' == '' ">17.0</Laerdal_Bindings_MacCatalyst___Sdk_Version> -->
<!-- <Laerdal_Bindings_MacCatalyst___Sdk_Version Condition=" '$(Laerdal_Bindings_MacCatalyst___Sdk_Version)' == '' ">18.0</Laerdal_Bindings_MacCatalyst___Sdk_Version> -->
<!-- <Laerdal_Bindings_MacCatalyst___Xcode_Ide_Dev_Path Condition=" '$(Laerdal_Bindings_MacCatalyst___Xcode_Ide_Dev_Path)' == '' ">/Applications/Xcode_15.4.app/Contents/Developer</Laerdal_Bindings_MacCatalyst___Xcode_Ide_Dev_Path> -->
</PropertyGroup>

Expand Down Expand Up @@ -77,10 +77,10 @@
<AllowedReferenceRelatedFileExtensions>$(AllowedReferenceRelatedFileExtensions);.pdb</AllowedReferenceRelatedFileExtensions>

<!-- these versions are getting replaced by the build script in one swift pass -->
<Version>1.0.1092.0</Version>
<FileVersion>1.0.1092.0</FileVersion>
<PackageVersion>1.0.1092.0</PackageVersion>
<AssemblyVersion>1.0.1092.0</AssemblyVersion>
<Version>1.0.1208.0</Version>
<FileVersion>1.0.1208.0</FileVersion>
<PackageVersion>1.0.1208.0</PackageVersion>
<AssemblyVersion>1.0.1208.0</AssemblyVersion>

<Title>$(PackageId)</Title>
<Summary>McuMgr Bindings for MacCatalyst - MAUI ready</Summary>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@
<AllowedReferenceRelatedFileExtensions>$(AllowedReferenceRelatedFileExtensions);.pdb</AllowedReferenceRelatedFileExtensions>

<!-- these versions are getting replaced by the build script in one swift pass -->
<Version>1.0.1092.0</Version>
<FileVersion>1.0.1092.0</FileVersion>
<PackageVersion>1.0.1092.0</PackageVersion>
<AssemblyVersion>1.0.1092.0</AssemblyVersion>
<Version>1.0.1208.0</Version>
<FileVersion>1.0.1208.0</FileVersion>
<PackageVersion>1.0.1208.0</PackageVersion>
<AssemblyVersion>1.0.1208.0</AssemblyVersion>

<Title>$(PackageId)</Title>
<Summary>McuMgr C# Implementation (WIP)</Summary>
Expand Down
10 changes: 5 additions & 5 deletions Laerdal.McuMgr.Bindings.iOS/Laerdal.McuMgr.Bindings.iOS.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
<NativeFrameworkActualTargetFolderpath>$(NativeFrameworkParentFolderpath)/McuMgrBindingsiOS.framework</NativeFrameworkActualTargetFolderpath>

<!-- set these explicitly here only for experimentation as far as cicd is concerned these values should be set in the .yml file of the pipeline -->
<!-- <Laerdal_Bindings_iOS___Sdk_Version Condition=" '$(Laerdal_Bindings_iOS___Sdk_Version)' == '' ">17.0</Laerdal_Bindings_iOS___Sdk_Version> -->
<!-- <Laerdal_Bindings_iOS___Sdk_Version Condition=" '$(Laerdal_Bindings_iOS___Sdk_Version)' == '' ">18.0</Laerdal_Bindings_iOS___Sdk_Version> -->
<!-- <Laerdal_Bindings_iOS___Xcode_Ide_Dev_Path Condition=" '$(Laerdal_Bindings_iOS___Xcode_Ide_Dev_Path)' == '' ">/Applications/Xcode_15.4.app/Contents/Developer/</Laerdal_Bindings_iOS___Xcode_Ide_Dev_Path> -->
</PropertyGroup>

Expand Down Expand Up @@ -74,10 +74,10 @@
<AllowedReferenceRelatedFileExtensions>$(AllowedReferenceRelatedFileExtensions);.pdb</AllowedReferenceRelatedFileExtensions>

<!-- these versions are getting replaced by the build script in one swift pass -->
<Version>1.0.1092.0</Version>
<FileVersion>1.0.1092.0</FileVersion>
<PackageVersion>1.0.1092.0</PackageVersion>
<AssemblyVersion>1.0.1092.0</AssemblyVersion>
<Version>1.0.1208.0</Version>
<FileVersion>1.0.1208.0</FileVersion>
<PackageVersion>1.0.1208.0</PackageVersion>
<AssemblyVersion>1.0.1208.0</AssemblyVersion>

<Title>$(PackageId)</Title>
<Summary>McuMgr Bindings for iOS - MAUI ready</Summary>
Expand Down
24 changes: 13 additions & 11 deletions Laerdal.McuMgr/Laerdal.McuMgr.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,13 @@
<!-- #1 these properties must be kept in sync between this file and laerdal.mcumgr.bindings.[android|ios|maccatalyst].csproj -->
<!-- #2 even though this is library is pure csharp we still have to specify the target-plaform-version for each target because if we dont dont the build system throws a tantrum from oct 2024 onwards -->
<TargetPlatformVersion Condition=" '$(IsIOS)' == 'true' and '$(Laerdal_Bindings_iOS___DotnetTargetPlatformVersion)' != '' ">$(Laerdal_Bindings_iOS___DotnetTargetPlatformVersion)</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsIOS)' == 'true' and '$(TargetPlatformVersion)' == '' ">17.0</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsAndroid)' == 'true' and '$(Laerdal_Bindings_Android___DotnetTargetPlatformVersion)' != '' ">$(Laerdal_Bindings_Android___DotnetTargetPlatformVersion)</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsMacCatalyst)' == 'true' and '$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)' != '' ">$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)</TargetPlatformVersion>

<TargetPlatformVersion Condition=" '$(IsIOS)' == 'true' and '$(TargetPlatformVersion)' == '' ">18.0</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsAndroid)' == 'true' and '$(TargetPlatformVersion)' == '' ">34</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsMacCatalyst)' == 'true' and '$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)' != '' ">$(Laerdal_Bindings_MacCatalyst___DotnetTargetPlatformVersion)</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsMacCatalyst)' == 'true' and '$(TargetPlatformVersion)' == '' ">17.0</TargetPlatformVersion>
<TargetPlatformVersion Condition=" '$(IsMacCatalyst)' == 'true' and '$(TargetPlatformVersion)' == '' ">18.0</TargetPlatformVersion>

<!-- minimum ios/android/maccatalyst os versions that we support -->
<SupportedOSPlatformVersion Condition=" '$(IsIOS)' == 'true' ">14.2</SupportedOSPlatformVersion>
<SupportedOSPlatformVersion Condition=" '$(IsAndroid)' == 'true' ">21</SupportedOSPlatformVersion>
Expand Down Expand Up @@ -71,10 +73,10 @@
<AllowedOutputExtensionsInPackageBuildOutputFolder>$(AllowedOutputExtensionsInPackageBuildOutputFolder);.pdb</AllowedOutputExtensionsInPackageBuildOutputFolder>

<!-- these versions are getting replaced by the build script in one swift pass -->
<Version>1.0.1092.0</Version>
<FileVersion>1.0.1092.0</FileVersion>
<PackageVersion>1.0.1092.0</PackageVersion>
<AssemblyVersion>1.0.1092.0</AssemblyVersion>
<Version>1.0.1208.0</Version>
<FileVersion>1.0.1208.0</FileVersion>
<PackageVersion>1.0.1208.0</PackageVersion>
<AssemblyVersion>1.0.1208.0</AssemblyVersion>

<Title>$(PackageId)</Title>
<Owners>$(Authors)</Owners>
Expand Down Expand Up @@ -167,21 +169,21 @@

<!-- ANDROID -->
<ItemGroup Condition=" '$(IsAndroid)' == 'true' ">
<PackageReference Include="Laerdal.McuMgr.Bindings.Android" Version="1.0.1092.0"/>
<PackageReference Include="Laerdal.McuMgr.Bindings.Android" Version="1.0.1208.0"/>
</ItemGroup>

<!-- IOS/MacCatalyst -->
<ItemGroup Condition=" '$(IsIOS)' == 'true' ">
<PackageReference Include="Laerdal.McuMgr.Bindings.iOS" Version="1.0.1092.0"/>
<PackageReference Include="Laerdal.McuMgr.Bindings.iOS" Version="1.0.1208.0"/>
</ItemGroup>

<ItemGroup Condition=" '$(IsNetXMacCatalyst)' == 'true' and '$(Should_Skip_MacCatalyst)' != 'true' ">
<PackageReference Include="Laerdal.McuMgr.Bindings.MacCatalyst" Version="1.0.1092.0"/>
<PackageReference Include="Laerdal.McuMgr.Bindings.MacCatalyst" Version="1.0.1208.0"/>
</ItemGroup>

<!-- NETSTANDARD -->
<ItemGroup Condition=" '$(IsDudNet8_0)' == 'true' ">
<PackageReference Include="Laerdal.McuMgr.Bindings.NetStandard" Version="1.0.1092.0"/>
<PackageReference Include="Laerdal.McuMgr.Bindings.NetStandard" Version="1.0.1208.0"/>
</ItemGroup>

</Project>

0 comments on commit 15b67a9

Please sign in to comment.