From: Mike McLaughlin <mikem@microsoft.com>
Date: Wed, 17 Jan 2024 01:41:35 +0000 (-0800)
Subject: Fix some packaging issues (#4465)
X-Git-Tag: accepted/tizen/unified/20241231.014852~40^2~231
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=17b9f21b18982e3d860ac7e9635e3da3dc89973a;p=platform%2Fcore%2Fdotnet%2Fdiagnostics.git

Fix some packaging issues (#4465)

Change SOS installer to copy subdirectories
---

diff --git a/src/SOS/SOS.InstallHelper/InstallHelper.cs b/src/SOS/SOS.InstallHelper/InstallHelper.cs
index 8073a9ced..ed72e4b8c 100644
--- a/src/SOS/SOS.InstallHelper/InstallHelper.cs
+++ b/src/SOS/SOS.InstallHelper/InstallHelper.cs
@@ -110,23 +110,11 @@ namespace SOS
 
                 // Copy native SOS files
                 WriteLine($"Copying files from {SOSNativeSourcePath}");
-                RetryOperation("Problem installing native SOS binaries", () => {
-                    foreach (string file in Directory.EnumerateFiles(SOSNativeSourcePath))
-                    {
-                        string destinationFile = Path.Combine(InstallLocation, Path.GetFileName(file));
-                        File.Copy(file, destinationFile, overwrite: true);
-                    }
-                });
+                RetryOperation("Problem installing native SOS binaries", () => CopyFiles(SOSNativeSourcePath, InstallLocation));
 
                 // Copy managed SOS files
                 WriteLine($"Copying files from {SOSManagedSourcePath}");
-                RetryOperation("Problem installing managed SOS binaries", () => {
-                    foreach (string file in Directory.EnumerateFiles(SOSManagedSourcePath))
-                    {
-                        string destinationFile = Path.Combine(InstallLocation, Path.GetFileName(file));
-                        File.Copy(file, destinationFile, overwrite: true);
-                    }
-                });
+                RetryOperation("Problem installing managed SOS binaries", () => CopyFiles(SOSManagedSourcePath, InstallLocation));
 
                 // Configure lldb
                 if (LLDBInitFile != null)
@@ -356,6 +344,21 @@ namespace SOS
             return $"{os}-{architectureString}";
         }
 
+        private static void CopyFiles(string sourcePath, string destinationPath)
+        {
+            foreach (string path in Directory.EnumerateDirectories(sourcePath))
+            {
+                string directory = Path.Combine(destinationPath, Path.GetFileName(path));
+                Directory.CreateDirectory(directory);
+                CopyFiles(path, directory);
+            }
+            foreach (string file in Directory.EnumerateFiles(sourcePath))
+            {
+                string destinationFile = Path.Combine(destinationPath, Path.GetFileName(file));
+                File.Copy(file, destinationFile, overwrite: true);
+            }
+        }
+
         private void WriteLine(string format, params object[] args)
         {
             m_writeLine?.Invoke(string.Format(format, args));
diff --git a/src/SOS/SOS.InstallHelper/SOS.InstallHelper.csproj b/src/SOS/SOS.InstallHelper/SOS.InstallHelper.csproj
index 9f007f064..315833e65 100644
--- a/src/SOS/SOS.InstallHelper/SOS.InstallHelper.csproj
+++ b/src/SOS/SOS.InstallHelper/SOS.InstallHelper.csproj
@@ -3,7 +3,7 @@
     <TargetFramework>netstandard2.0</TargetFramework>
     <AssemblyName>SOS.InstallHelper</AssemblyName>
     <NoWarn>;1591;1701</NoWarn>
-    <IsPackable>false</IsPackable>
+    <IsPackable>true</IsPackable>
     <Description>Diagnostic SOS Install Helper</Description>
     <PackageReleaseNotes>$(Description)</PackageReleaseNotes>
     <PackageTags>SOS</PackageTags>
diff --git a/src/SOS/SOS.Package/SOS.Package.csproj b/src/SOS/SOS.Package/SOS.Package.csproj
index 8ae407ed7..5fe7727d8 100644
--- a/src/SOS/SOS.Package/SOS.Package.csproj
+++ b/src/SOS/SOS.Package/SOS.Package.csproj
@@ -10,7 +10,7 @@
     <IncludeBuildOutput>false</IncludeBuildOutput>
     <SOSPackagePathPrefix>tools</SOSPackagePathPrefix>
     <GalleryManifestName>$(ArtifactsPackagesDir)\GalleryManifest.xml</GalleryManifestName>
-    <BeforePack>GenerateGalleryZip;GenerateSymbolsZip</BeforePack>
+    <BeforePack>GenerateGalleryManifest</BeforePack>
     <IsShipping>true</IsShipping>
     <IsShippingPackage>false</IsShippingPackage>
   </PropertyGroup>
@@ -77,93 +77,4 @@
     </ItemGroup>
   </Target>
 
-  <Target Name="GenerateGalleryZip" DependsOnTargets="GenerateGalleryManifest">
-    <PropertyGroup>
-      <GalleryDir>$(ArtifactsBinDir)\gallery</GalleryDir>
-      <GallerySubDir>$(GalleryDir)\$(FileVersion)</GallerySubDir>
-    </PropertyGroup>
-
-    <ItemGroup>
-      <ZipSourceFiles Include="$(GalleryManifestName)">
-         <TargetPath>$(GalleryDir)</TargetPath>
-      </ZipSourceFiles>
-
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.x64.$(Configuration)\sos.dll">
-         <TargetPath>$(GallerySubDir)\win-x64</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.x64.$(Configuration)\Microsoft.DiaSymReader.Native.amd64.dll">
-         <TargetPath>$(GallerySubDir)\win-x64</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(SOSExtensionsBinaries)">
-         <TargetPath>$(GallerySubDir)\win-x64</TargetPath>
-      </ZipSourceFiles>
-
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.x86.$(Configuration)\sos.dll">
-         <TargetPath>$(GallerySubDir)\win-x86</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.x86.$(Configuration)\Microsoft.DiaSymReader.Native.x86.dll">
-         <TargetPath>$(GallerySubDir)\win-x86</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(SOSExtensionsBinaries)">
-         <TargetPath>$(GallerySubDir)\win-x86</TargetPath>
-      </ZipSourceFiles>
-
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.arm.$(Configuration)\sos.dll">
-         <TargetPath>$(GallerySubDir)\win-arm</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.arm.$(Configuration)\Microsoft.DiaSymReader.Native.arm.dll">
-         <TargetPath>$(GallerySubDir)\win-arm</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(SOSExtensionsBinaries)">
-         <TargetPath>$(GallerySubDir)\win-arm</TargetPath>
-      </ZipSourceFiles>
-
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.arm64.$(Configuration)\sos.dll">
-         <TargetPath>$(GallerySubDir)\win-arm64</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(ArtifactsBinDir)Windows_NT.arm64.$(Configuration)\Microsoft.DiaSymReader.Native.arm64.dll">
-         <TargetPath>$(GallerySubDir)\win-arm64</TargetPath>
-      </ZipSourceFiles>
-      <ZipSourceFiles Include="$(SOSExtensionsBinaries)">
-         <TargetPath>$(GallerySubDir)\win-arm64</TargetPath>
-      </ZipSourceFiles>
-    </ItemGroup>
-
-    <Copy SourceFiles="@(ZipSourceFiles)" DestinationFolder="%(ZipSourceFiles.TargetPath)" />
-
-    <ZipDirectory DestinationFile="$(ArtifactsPackagesDir)$(FileVersion).zip" SourceDirectory="$(GalleryDir)" Overwrite="true" />
-  </Target>
-
-  <Target Name="GenerateSymbolsZip">
-    <PropertyGroup>
-      <SymbolsDir>$(ArtifactsBinDir)\symbols</SymbolsDir>
-    </PropertyGroup>
-
-    <ItemGroup>
-      <ZipSymbolFiles Include="$(ArtifactsSymStoreDirectory)\SOS.Extensions\netstandard2.0\publish\*.pdb">
-         <TargetPath>$(SymbolsDir)</TargetPath>
-      </ZipSymbolFiles>
-
-      <ZipSymbolFiles Include="$(ArtifactsBinDir)Windows_NT.x64.$(Configuration)\PDB\sos.pdb">
-         <TargetPath>$(SymbolsDir)\win-x64</TargetPath>
-      </ZipSymbolFiles>
-
-      <ZipSymbolFiles Include="$(ArtifactsBinDir)Windows_NT.x86.$(Configuration)\PDB\sos.pdb">
-         <TargetPath>$(SymbolsDir)\win-x86</TargetPath>
-      </ZipSymbolFiles>
-
-      <ZipSymbolFiles Include="$(ArtifactsBinDir)Windows_NT.arm.$(Configuration)\PDB\sos.pdb">
-         <TargetPath>$(SymbolsDir)\win-arm</TargetPath>
-      </ZipSymbolFiles>
-
-      <ZipSymbolFiles Include="$(ArtifactsBinDir)Windows_NT.arm64.$(Configuration)\PDB\sos.pdb">
-         <TargetPath>$(SymbolsDir)\win-arm64</TargetPath>
-      </ZipSymbolFiles>
-    </ItemGroup>
-
-    <Copy SourceFiles="@(ZipSymbolFiles)" DestinationFolder="%(ZipSymbolFiles.TargetPath)" />
-
-    <ZipDirectory DestinationFile="$(ArtifactsPackagesDir)$(FileVersion).symbols.zip" SourceDirectory="$(SymbolsDir)" Overwrite="true" />
-  </Target>
-
 </Project>
diff --git a/src/SOS/SOS.Package/SOS.Symbol.Package.csproj b/src/SOS/SOS.Package/SOS.Symbol.Package.csproj
index 83cd791cf..59adf253d 100644
--- a/src/SOS/SOS.Package/SOS.Symbol.Package.csproj
+++ b/src/SOS/SOS.Package/SOS.Symbol.Package.csproj
@@ -15,7 +15,7 @@
 
   <ItemGroup>
     <None Include="$(ArtifactsBinDir)\SOS.Extensions\$(Configuration)\netstandard2.0\publish\*.pdb" Pack="true" Visible="false">
-      <PackagePath>$(SOSPackagePathPrefix)</PackagePath>
+      <PackagePath>$(SOSPackagePathPrefix)/lib</PackagePath>
     </None>
 
     <None Include="$(ArtifactsBinDir)\Windows_NT.x64.$(Configuration)\PDB\sos.pdb" Pack="true" Visible="false">