Fix inconsistent System.Runtime.CompilerServices.Unsafe versions.

ILSpy.ReadyToRun/ILSpy.ReadyToRun.csproj

@@ -73,9 +73,18 @@
     <Page Include="ReadyToRunOptionPage.xaml" />
   </ItemGroup>
 
+  <Import Project="../packages.props"/>
+
   <ItemGroup>
     <PackageReference Include="Iced" Version="1.8.0" />
     <PackageReference Include="ILCompiler.Reflection.ReadyToRun" Version="1.0.11-alpha" />
+    <!-- ILCompiler.Reflection.ReadyToRun has dependencies on System.Reflection.Metadata and
+         System.Runtime.CompilerServices.Unsafe. Because the AddIn compiles into ILSpy's output
+         directory, we're at risk of overwriting our dependencies with different versions.
+         So ensure NuGet uses consistent versions (from our packages.props) for these.
+         -->
+    <PackageReference Include="System.Reflection.Metadata" Version="$(SystemReflectionMetadataVersion)" />
+    <PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="$(SystemCompilerServicesUnsafeVersion)" />
   </ItemGroup>
 
   <Target Name="RemoveTransitiveProjectReferences" AfterTargets="IncludeTransitiveProjectReferences">

ILSpy/ILSpy.csproj

@@ -52,6 +52,8 @@
 
   <ItemGroup>
     <PackageReference Include="AvalonEdit" Version="$(AvalonEditVersion)" />
+    <PackageReference Include="System.Reflection.Metadata" Version="$(SystemReflectionMetadataVersion)" />
+    <PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="$(SystemCompilerServicesUnsafeVersion)" />
     <PackageReference Include="Dirkster.AvalonDock" Version="3.6.1" />
     <PackageReference Include="Microsoft.VisualStudio.Composition" Version="16.3.7" />
     <PackageReference Include="System.Composition" Version="1.3.0" />

ILSpy/Properties/app.config.template

@@ -32,6 +32,10 @@
 				<assemblyIdentity name="System.Reflection.Metadata" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
 				<bindingRedirect oldVersion="0.0.0.0-5.0.0.0" newVersion="5.0.0.0" />
 			</dependentAssembly>
+			<dependentAssembly>
+				<assemblyIdentity name="System.Runtime.CompilerServices.Unsafe" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
+				<bindingRedirect oldVersion="0.0.0.0-4.0.6.0" newVersion="4.0.6.0" />
+			</dependentAssembly>
 		</assemblyBinding>
 	</runtime>
 </configuration>

packages.props

@@ -1,10 +1,13 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project>
-  <!-- Centrally define the versions of the NuGet packages we use -->
+  <!-- Centrally define the versions of the NuGet packages we use.
+       Note: when updating these, ensure to also adjust the binding redirects in app.config.template appropriately.
+   -->
   <PropertyGroup>
     <HumanizerVersion>2.2.0</HumanizerVersion>
     <SystemCollectionsImmutableVersion>5.0.0-preview.8.20407.11</SystemCollectionsImmutableVersion>
     <SystemReflectionMetadataVersion>5.0.0-preview.8.20407.11</SystemReflectionMetadataVersion>
+    <SystemCompilerServicesUnsafeVersion>4.7.1</SystemCompilerServicesUnsafeVersion>
     <ILAsmVersion>5.0.0-rc.2.20475.5</ILAsmVersion>  <!-- Microsoft.NETCore.ILAsm -->
     <RoslynVersion>3.8.0-4.final</RoslynVersion>     <!-- Microsoft.CodeAnalysis.* -->
     <MonoCecilVersion>0.10.3</MonoCecilVersion>