From f06f7c1f83e6c5c8a72a79560a6ae76e0edfc386 Mon Sep 17 00:00:00 2001 From: Jonathan Peppers Date: Mon, 4 May 2026 13:41:41 -0500 Subject: [PATCH] Fix NonShipping casing for Xamarin.Android.Tools.AndroidSdk package The ManifestArtifactData had "Nonshipping=true" (lowercase s), but Arcade's ArtifactModel.NonShipping property looks up "NonShipping" (capital S). During build, this worked because MSBuildListSplitter creates a case-insensitive dictionary. However, when the manifest XML is parsed back (by PublishBuildAssets or darc gather-drop), CreateAttributeDictionary() creates a case-sensitive dictionary, so the lookup fails and defaults to false (shipping). Also removed the unused IsShipping="false" metadata, which is not read by PackageArtifactModelFactory. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- build-tools/create-packs/Directory.Build.targets | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build-tools/create-packs/Directory.Build.targets b/build-tools/create-packs/Directory.Build.targets index c24a926da19..67acf8b5aa9 100644 --- a/build-tools/create-packs/Directory.Build.targets +++ b/build-tools/create-packs/Directory.Build.targets @@ -167,7 +167,7 @@ - +