Skip to content

Conversation

@dellis1972
Copy link
Contributor

@dellis1972 dellis1972 commented Jan 25, 2022

We are seeing the following error on some builds

error XABBA7000: Xamarin.Tools.Zip.ZipException: Zip archive inconsistent: entry 0: invalid WinZip AES extra field

Verison 2.0.2 of LibZipSharp was supposed to fix this, but it had a bug.
Which would cause corruption in the zip file itself.

This bug was fixed in 2.0.3 of LibZipSharp. So lets bump to that.

@dellis1972 dellis1972 marked this pull request as ready for review January 26, 2022 16:47
@dellis1972 dellis1972 changed the title Investigate BuildApk error WIP Investigate BuildApk error Jan 26, 2022
@dellis1972 dellis1972 changed the title Investigate BuildApk error Bump LibZipSharp to 2.0.3 Jan 26, 2022
@jonpryor
Copy link
Contributor

The AndroidManifestProperties("3.1.3a1","42","") test failed, which is slightly concerning:

                   Task "AndroidZipAlign" (TaskId:308)
                     Task Parameter:DestinationDirectory=bin/Debug/ (TaskId:308)
                     Task Parameter:Source=obj/Debug/android/bin/com.xamarin.singleproject.apk (TaskId:308)
                     Task Parameter:ToolPath=/Users/runner/Library/Android/sdk/build-tools/30.0.3 (TaskId:308)
                     /Users/runner/Library/Android/sdk/build-tools/30.0.3/zipalign -p 4 "obj/Debug/android/bin/com.xamarin.singleproject.apk" "bin/Debug//com.xamarin.singleproject-Signed.apk"  (TaskId:308)
11:42:47.203   1:7>/Users/runner/Library/Android/dotnet/packs/Microsoft.Android.Sdk.Darwin/31.0.200-ci.pr.gh6667.43/tools/Xamarin.Android.Common.targets(2292,2): error ANDZA0000: Unable to open 'obj/Debug/android/bin/com.xamarin.singleproject.apk' as zip archive [/Users/runner/work/1/a/TestRelease/01-26_10.55.27/temp/AndroidManifestProperties3.1.3a142/UnnamedProject.csproj]
                     Unable to open 'obj/Debug/android/bin/com.xamarin.singleproject.apk' as zip archive (TaskId:308)

It looks like that file should exist:

                   Task "Copy" (TaskId:301)
                     Task Parameter:DestinationFolder=bin/Debug/ (TaskId:301)
                     Task Parameter:SourceFiles=/Users/runner/work/1/a/TestRelease/01-26_10.55.27/temp/AndroidManifestProperties3.1.3a142/obj/Debug/android/bin/com.xamarin.singleproject.apk (TaskId:301)
                     Copying file from "/Users/runner/work/1/a/TestRelease/01-26_10.55.27/temp/AndroidManifestProperties3.1.3a142/obj/Debug/android/bin/com.xamarin.singleproject.apk" to "/Users/runner/work/1/a/TestRelease/01-26_10.55.27/temp/AndroidManifestProperties3.1.3a142/bin/Debug/com.xamarin.singleproject.apk". (TaskId:301)
                   Done executing task "Copy". (TaskId:301)

Restarting the tests.

@jonpryor
Copy link
Contributor

Looks like the restart "fixed" the AndroidManifestProperties("3.1.3a1","42","") test.

@jonpryor jonpryor merged commit 23a63c5 into dotnet:main Jan 28, 2022
@dellis1972 dellis1972 deleted the buildapkbug branch January 28, 2022 09:52
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
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.

2 participants