sourcelink it is

StackExchange · Dec 14, 2018 · d595fac · d595fac
1 parent c6a151e
commit d595fac
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -18,6 +18,7 @@
 Version 5.1.0
 * updating roslyn packages to 2.10.0
 * updating mvc packages to 5.2.7
+* fixing SourceLink support for native pdbs
 
 Version 5.0.0
 * updating how Precompilation.Build resolves it's Workspaces dependency

diff --git a/StackExchange.Precompilation.Build/StackExchange.Precompilation.Build.csproj b/StackExchange.Precompilation.Build/StackExchange.Precompilation.Build.csproj
@@ -17,6 +17,7 @@
       <Name>StackExchange.Precompilation</Name>
     </ProjectReference>
     <PackageReference Include="Microsoft.CodeAnalysis.CSharp.Workspaces" Version="2.10.0" PrivateAssets="all" />
+    <PackageReference Include="Microsoft.DiaSymReader.Native" Version="1.7.0" PrivateAssets="all" />
   </ItemGroup>
 
   <Target Name="_ToolsSetup">

diff --git a/StackExchange.Precompilation.Build/StackExchange.Precompilation.Build.packages.config b/StackExchange.Precompilation.Build/StackExchange.Precompilation.Build.packages.config
@@ -0,0 +1,7 @@
+  <!--
+    HACK
+
+    We need to include the native DiaSymReader Native runtime files in order to support sourcelink etc.
+    Roslyn does it too: https://github.com/dotnet/roslyn/blob/1fe72cd3f196005a1a407c582f60145d096b2f43/src/Compilers/Core/Portable/Microsoft.CodeAnalysis.csproj#L26-L37
+    We make build/Microsoft.DiaSymRerader.Native.props from the package do it, via the existance of this dummy packages.config file.
+   -->