Add category metadata to ManifestArtifactData metadata instead of Category metadata...
authorJustin Anderson <jander-msft@users.noreply.github.com>
Wed, 17 Jun 2020 18:38:33 +0000 (11:38 -0700)
committerGitHub <noreply@github.com>
Wed, 17 Jun 2020 18:38:33 +0000 (11:38 -0700)
eng/Publishing.props

index 144c15536765380a85386fc929d6a37b99ce380e..cf9d9e5a6940e187523dcab5ae0479cc38b234f4 100644 (file)
   <Target Name="CollectPackageArtifactFiles"
           Inputs="@(PackageFile)"
           Outputs="%(PackageFile.Identity).notexist">
+
     <!-- Find the artifact files next to the package file. -->
     <PropertyGroup>
       <_BlobGroupFilePath>%(PackageFile.FullPath).blobgroup</_BlobGroupFilePath>
       <_ChecksumFilePath>%(PackageFile.FullPath).sha512</_ChecksumFilePath>
     </PropertyGroup>    
+
     <!-- Read in blob group name, if it exists -->
     <ReadLinesFromFile File="$(_BlobGroupFilePath)" Condition="Exists('$(_BlobGroupFilePath)')">
       <Output TaskParameter="Lines" PropertyName="_BlobGroupName"/>
     </ReadLinesFromFile>
+
+    <!-- Calculate manifest artifact data for each file type. -->
+    <ItemGroup>
+      <_CommonArtifactData Include="NonShipping=true" Condition="'%(PackageFile.IsShipping)' != 'true'" />
+    </ItemGroup>
     <ItemGroup>
-      <!-- Capture package; setting cateogry to Other will upload to installers blob feed. -->
-      <_BlobItem Include="%(PackageFile.FullPath)" Category="Other" />
-      <!-- Capture checksum -->
-      <_BlobItem Include="$(_ChecksumFilePath)" Condition="Exists('$(_ChecksumFilePath)')" />
+      <_PackageArtifactData Include="@(_CommonArtifactData)" />
+      <!-- Setting Category to Other will upload to installers blob feed. -->
+      <_PackageArtifactData Include="Category=Other" />
     </ItemGroup>
+
+    <!-- Capture each blob item to upload to blob feed -->
     <ItemGroup>
-      <!-- Add artifact items to be pushed to blob feed -->
+      <_BlobItem Include="%(PackageFile.FullPath)">
+        <ManifestArtifactData>@(_PackageArtifactData)</ManifestArtifactData>
+      </_BlobItem>
+      <_BlobItem Include="$(_ChecksumFilePath)" Condition="Exists('$(_ChecksumFilePath)')">
+        <ManifestArtifactData Condition="'@(_CommonArtifactData)' != ''">@(_CommonArtifactData)</ManifestArtifactData>
+      </_BlobItem>
+    </ItemGroup>
+
+    <!-- Add artifact items to be pushed to blob feed -->
+    <ItemGroup>      
       <ItemsToPushToBlobFeed Include="@(_BlobItem)" Condition="'$(_BlobGroupName)' != ''">
         <RelativeBlobPath>diagnostics/$(_BlobGroupName)/%(_BlobItem.Filename)%(_BlobItem.Extension)</RelativeBlobPath>
-        <ManifestArtifactData Condition="'%(PackageFile.IsShipping)' != 'true'">NonShipping=true</ManifestArtifactData>
         <PublishFlatContainer>true</PublishFlatContainer>
       </ItemsToPushToBlobFeed>
     </ItemGroup>