-
Notifications
You must be signed in to change notification settings - Fork 555
[XABT] remove $(_AndroidUseLibZipSharp)=true
default value
#9840
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
Conversation
I believe this has a new issue now: |
Context: dotnet/runtime#112017 In .NET 10 Preview 1, when using System.IO.Compression to create `.apk` files, `zipalign` was giving the error: 01-30 21:38:27.669 38611 159726 W zip : WARNING: header mismatch To workaround, we temporarily set `$(_AndroidUseLibZipSharp)=true`. We think this is fixed now, so partially revert f3ef4fe.
b71bce5
to
ac295a0
Compare
Update: We're hoping this will be resolved with the latest fixes:
We also added async support if you have a chance to test it:
|
So far this looks good, the MSBuild tests build many projects using I restarted a few of the lanes that the Android emulator timed out. |
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.
🥳 Yay! Thank you everyone for helping us ship this performance win for our users!
The test lanes I reran are now failing due to something with AzDO downloading secrets:
I think this is safe to merge, because 100s of tests were failing when this wasn't working. We can revisit if something comes up. |
Context: dotnet/runtime#112017
Context: dotnet/runtime#113306
In .NET 10 Preview 1, when using System.IO.Compression to create
.apk
files,zipalign
was giving the error:To workaround, we temporarily set
$(_AndroidUseLibZipSharp)=true
.We think this is fixed now, so partially revert f3ef4fe.