[PERF][MAUI] Add ios-arm64 logic to create .ipa for perf runs #66159
Conversation
|
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
ghost
assigned
Switches from simulator builds to device builds so we can track the size for the .ipa. We generate a dummy codesigning cert and provisioning profile since device builds require that.
| # remove net6.0-maccatalyst to work around https://github.com/dotnet/sdk/issues/21877 | ||
| cp MauiTesting.csproj MauiTesting.csproj.bak | ||
| sed -i'' -e 's/net6.0-ios;net6.0-maccatalyst/net6.0-ios/g' MauiTesting.csproj | ||
| ../dotnet publish -bl:MauiiOS.binlog -f net6.0-ios --self-contained -r ios-arm64 -c Release /p:_RequireCodeSigning=false |
There was a problem hiding this comment.
Instead of using sed, I think you can use a couple different commands mentioned here: dotnet/sdk#21877 (comment)
> dotnet restore
> dotnet build --no-restore -p:BuildIpa=true ...
They are fixing this, so maybe somewhat soon you won't need to workaround it?
There was a problem hiding this comment.
I'd prefer not to use dotnet build with some arbitrary properties set since we should really be testing publish here like a regular customer would. We can remove the workaround once we have an SDK with the fix.
There was a problem hiding this comment.
On the Android side, I wouldn't even recommend using publish. It basically just copies extra files.
|
@akoeplinger I think we can close this. Changes were merged as a part of #66358 |
3 participants
Switches from simulator builds to device builds so we can track the size for the .ipa.
We generate a dummy codesigning cert and provisioning profile since device builds require that.