Skip to content

[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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

jonathanpeppers
Copy link
Member

@jonathanpeppers jonathanpeppers commented Feb 25, 2025

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:

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.

@jonathanpeppers
Copy link
Member Author

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.
@jonathanpeppers jonathanpeppers force-pushed the dev/peppers/_AndroidUseLibZipSharp branch from b71bce5 to ac295a0 Compare April 17, 2025 15:34
@carlossanlop
Copy link

carlossanlop commented Apr 17, 2025

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:

@jonathanpeppers
Copy link
Member Author

jonathanpeppers commented Apr 17, 2025

So far this looks good, the MSBuild tests build many projects using System.IO.Compression.ZipArchive.

I restarted a few of the lanes that the Android emulator timed out.

@jonathanpeppers jonathanpeppers marked this pull request as ready for review April 17, 2025 20:00
@jonathanpeppers jonathanpeppers requested a review from jpobst April 17, 2025 20:00
Copy link
Contributor

@jpobst jpobst left a 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!

@jonathanpeppers
Copy link
Member Author

The test lanes I reran are now failing due to something with AzDO downloading secrets:

##[error] developer-id-installer: "getaddrinfo ENOTFOUND xamarin-releng.vault.azure.net"

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.

@jonathanpeppers jonathanpeppers merged commit 1be3e2a into main Apr 17, 2025
55 of 59 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/peppers/_AndroidUseLibZipSharp branch April 17, 2025 21:01
@github-actions github-actions bot locked and limited conversation to collaborators May 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants