-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Update Maui Workload Install Process for Perf #64692
Conversation
…test if new dotnet version can be installed similar to how it is done in the Maui repo.
…r workload install.
Tagging subscribers to this area: @hoyosjs Issue DetailsFixes dotnet/performance#2230, where an error with installing the latest Maui workload would occur but not error out the pipeline. This fixes that by installing the latest 6.0.2xx dotnet sdk for installing the workload and building the Maui apps.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Perf run succeeded where expected on a private run, merging. |
curl -o dotnet-install.sh 'https://dotnet.microsoft.com/download/dotnet/scripts/v1/dotnet-install.sh' | ||
chmod -R a+rx . | ||
./dotnet-install.sh --channel 6.0.2xx --quality signed --install-dir . | ||
./dotnet --info |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this line for? Is it needed?
mv ./bin/Release/net6.0-android/com.companyname.MauiTesting-Signed.apk ./MauiAndroidDefault.apk | ||
displayName: Build MAUI Android | ||
workingDirectory: $(Build.SourcesDirectory)/MauiTesting | ||
|
||
- script: | | ||
chmod -R a+r . | ||
../dotnet.sh publish -bl:MauiiOS.binlog -f net6.0-ios -c Release | ||
../dotnet build -bl:MauiiOS.binlog -f net6.0-ios -c Release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What I've found to work is:
- Remove the
net6.0-maccatalyst
TFM from theTargetFrameworks
property in the .csproj. dotnet publish -c Release -f net6.0-ios -r ios-arm64
I've logged xamarin/xamarin-macios#14015 for this, which was closed in favor of dotnet/sdk#21877.
mv ./bin/Release/net6.0-ios/iossimulator-x64/MauiTesting.app ./MauiiOSDefault.app | ||
displayName: Build MAUI iOS | ||
workingDirectory: $(Build.SourcesDirectory)/MauiTesting | ||
|
||
- script: | | ||
chmod -R a+r . | ||
../dotnet.sh publish -bl:MauiMacCatalyst.binlog -f net6.0-maccatalyst -c Release | ||
../dotnet publish -bl:MauiMacCatalyst.binlog -f net6.0-maccatalyst -c Release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we care about maccatalyst size? I'm not certain this is an important scenario to track. It may just be wasting resources and cluttering the perf dashboard.
* Add dotnet install and run to test if new dotnet version can be installed similar to how it is done in the Maui repo. * Download and use local dotnet version. * Use updated dotnet for more of the pipeline. * Test with made runtime packs and and add ios arm pack substitution. * Try building instead of publishing. * Reenable everything for full test run. * Package name now seems to default to lower case, at least for maui.
* Add dotnet install and run to test if new dotnet version can be installed similar to how it is done in the Maui repo. * Download and use local dotnet version. * Use updated dotnet for more of the pipeline. * Test with made runtime packs and and add ios arm pack substitution. * Try building instead of publishing. * Reenable everything for full test run. * Package name now seems to default to lower case, at least for maui.
Fixes dotnet/performance#2230, where an error with installing the latest Maui workload would occur but not error out the pipeline. This fixes that by installing the latest 6.0.2xx dotnet sdk for installing the workload and building the Maui apps. Includes some minor name fixes and adds ios runtime building.