Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop using FileSystem and Common submodules #4536

Merged
merged 6 commits into from
Apr 21, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 0 additions & 8 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,11 +1,3 @@
[submodule "submodules/FileSystem"]
jeffkl marked this conversation as resolved.
Show resolved Hide resolved
path = submodules/FileSystem
url = https://github.com/NuGet/FileSystem.git
branch = NuGet/SubModuleIntegration
[submodule "submodules/Common"]
path = submodules/Common
url = https://github.com/aspnet/Common.git
branch = NuGet/SubModuleIntegration
[submodule "submodules/NuGet.Build.Localization"]
path = submodules/NuGet.Build.Localization
url = https://github.com/NuGet/NuGet.Build.Localization.git
2 changes: 1 addition & 1 deletion build/bootstrap.proj
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
<PackageDownload Include="Microsoft.Web.Xdt" version="[3.0.0]" />
<PackageDownload Include="Newtonsoft.Json" version="[13.0.1]" />
<PackageDownload Include="NuGet.Client.EndToEnd.TestData" version="[1.0.0]" />
<PackageDownload Include="NuGetValidator" version="[2.0.3]" />
<PackageDownload Include="NuGetValidator" version="[2.0.5]" />
</ItemGroup>

</Project>
3 changes: 3 additions & 0 deletions build/packages.targets
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@
<PackageReference Update="Microsoft.CodeAnalysis.FxCopAnalyzers" Version="2.9.8" />
<PackageReference Update="Microsoft.DataAI.NuGetRecommender.Contracts" Version="2.1.0" />
<PackageReference Update="Microsoft.Extensions.CommandLineUtils.Sources" Version="$(MicrosoftExtensionsCommandLineUtilsSourcesPackageVersion)" />
<PackageReference Update="Microsoft.Extensions.FileSystemGlobbing" Version="5.0.0"/>
<PackageReference Update="Microsoft.Extensions.FileProviders.Abstractions" Version="5.0.0" />
<PackageReference Update="Microsoft.Internal.VisualStudio.Shell.Framework" Version="$(VSFrameworkVersion)" />
<PackageReference Update="Microsoft.SourceLink.GitHub" Version="1.0.0" />
<PackageReference Update="Microsoft.TeamFoundationServer.ExtendedClient" Version="$(VSServicesVersion)" />
Expand All @@ -50,6 +52,7 @@
<PackageReference Update="Newtonsoft.Json" Version="$(NewtonsoftJsonPackageVersion)" />
<PackageReference Update="SharpZipLib" Version="1.3.3" />
<PackageReference Update="System.ComponentModel.Composition" Version="$(SystemComponentModelCompositionPackageVersion)" />
<PackageReference Update="System.Runtime.Serialization.Primitives" Version="$(SystemPackagesVersion)" />
<!--
The Microsoft.VisualStudio.SDK metapackage brings in System.Threading.Tasks.Dataflow 4.11.1 (assembly version 4.9.5.0).
However, our MSBuild integration tests use Microsoft.Build 16.8.0, which requires System.Threading.Tasks.Dataflow 4.9.0 (assembly version 4.9.3.0).
Expand Down
2 changes: 1 addition & 1 deletion eng/pipelines/templates/Build_and_UnitTest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@ steps:
inputs:
artifactName: LocValidationLogs
targetPath: "$(Build.Repository.LocalPath)\\logs\\LocalizationValidation"
condition: "and(succeeded(), eq(variables['BuildRTM'], 'false'))"
condition: "and(succeededOrFailed(), eq(variables['BuildRTM'], 'false'))"

# Use dotnet msbuild instead of MSBuild CLI.
# Using MSBuild CLI results in an assembly binding failure for NuGet.Common.dll 4.4.0.3 because Microsoft.DotNet.Build.Tasks.Feed.dll references SleetLib.dll which references NuGet.Common 4.4.0.3.
Expand Down
2 changes: 1 addition & 1 deletion scripts/cibuild/BuildValidator.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ param

if ($BuildRTM -eq 'false')
{
$NuGetValidator = [System.IO.Path]::Combine($RepoRoot, 'packages', 'nugetvalidator', '2.0.3', 'tools', 'NuGetValidator.exe')
$NuGetValidator = [System.IO.Path]::Combine($RepoRoot, 'packages', 'nugetvalidator', '2.0.5', 'tools', 'NuGetValidator.exe')
$LocalizationRepository = [System.IO.Path]::Combine($RepoRoot, 'submodules', 'NuGet.Build.Localization', 'localize', 'comments', '15')

if ($ValidateVsix)
Expand Down
3 changes: 3 additions & 0 deletions setup/Microsoft.VisualStudio.NuGet.BuildTools.swr
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ folder InstallDir:\Common7\IDE\CommonExtensions\Microsoft\NuGet
file source=$(NuGetTargetsBasePath)NuGet.targets
file source=$(NuGetTargetsBasePath)NuGet.props
file source=$(NuGetTargetsBasePath)NuGet.RestoreEx.targets
file source=$(ReferenceOutputPath)Microsoft.Extensions.FileProviders.Abstractions.dll
file source=$(ReferenceOutputPath)Microsoft.Extensions.FileSystemGlobbing.dll
file source=$(ReferenceOutputPath)Microsoft.Extensions.Primitives.dll
file source=$(ReferenceOutputPath)Microsoft.Web.XmlTransform.dll
file source=$(ReferenceOutputPath)Microsoft.Build.NuGetSdkResolver.dll
file source=$(NewtonsoftJsonPath)Newtonsoft.Json.dll
Expand Down
7 changes: 7 additions & 0 deletions src/NuGet.Clients/NuGet.CommandLine/ilmerge.props
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="15.0">
<ItemGroup>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.FileProviders.Abstractions.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.FileSystemGlobbing.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.Primitives.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Web.XmlTransform.dll"/>
<MergeInclude Include="$(OutputPath)Newtonsoft.Json.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.Build.Tasks.dll"/>
Expand All @@ -18,11 +21,13 @@
<MergeInclude Include="$(OutputPath)NuGet.Resolver.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.Versioning.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.Core.dll"/>
<MergeInclude Include="$(OutputPath)System.Memory.dll"/>
<MergeInclude Include="$(OutputPath)System.Threading.Tasks.Dataflow.dll"/>

<MergeExclude Include="$(OutputPath)Microsoft.VisualStudio.Setup.Configuration.Interop.dll"/>
<MergeExclude Include="$(OutputPath)Microsoft.CSharp.dll"/>
<MergeExclude Include="$(OutputPath)mscorlib.dll"/>
<MergeExclude Include="$(OutputPath)System.Buffers.dll"/>
<MergeExclude Include="$(OutputPath)System.Collections.Concurrent.dll"/>
<MergeExclude Include="$(OutputPath)System.Collections.dll"/>
<MergeExclude Include="$(OutputPath)System.ComponentModel.Composition.dll"/>
Expand All @@ -32,8 +37,10 @@
<MergeExclude Include="$(OutputPath)System.IO.Compression.dll"/>
<MergeExclude Include="$(OutputPath)System.Net.Http.dll"/>
<MergeExclude Include="$(OutputPath)System.Net.Http.WebRequest.dll"/>
<MergeExclude Include="$(OutputPath)System.Numerics.Vectors.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.Serialization.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.CompilerServices.Unsafe.dll"/>
<MergeExclude Include="$(OutputPath)System.Security.dll"/>
<MergeExclude Include="$(OutputPath)System.ServiceModel.dll"/>
<MergeExclude Include="$(OutputPath)System.Xml.dll"/>
Expand Down
7 changes: 7 additions & 0 deletions src/NuGet.Clients/NuGet.MSSigning.Extensions/ilmerge.props
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="15.0">
<ItemGroup>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.FileProviders.Abstractions.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.FileSystemGlobbing.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Extensions.Primitives.dll"/>
<MergeInclude Include="$(OutputPath)Microsoft.Web.XmlTransform.dll"/>
<MergeInclude Include="$(OutputPath)Newtonsoft.Json.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.Build.Tasks.dll"/>
Expand All @@ -19,11 +22,13 @@
<MergeInclude Include="$(OutputPath)NuGet.Versioning.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.Core.dll"/>
<MergeInclude Include="$(OutputPath)NuGet.MSSigning.Extensions.dll"/>
<MergeInclude Include="$(OutputPath)System.Memory.dll"/>
<MergeInclude Include="$(OutputPath)System.Threading.Tasks.Dataflow.dll"/>

<MergeExclude Include="$(OutputPath)Microsoft.VisualStudio.Setup.Configuration.Interop.dll"/>
<MergeExclude Include="$(OutputPath)Microsoft.CSharp.dll"/>
<MergeExclude Include="$(OutputPath)mscorlib.dll"/>
<MergeExclude Include="$(OutputPath)System.Buffers.dll"/>
<MergeExclude Include="$(OutputPath)System.Collections.Concurrent.dll"/>
<MergeExclude Include="$(OutputPath)System.Collections.dll"/>
<MergeExclude Include="$(OutputPath)System.ComponentModel.Composition.dll"/>
Expand All @@ -33,8 +38,10 @@
<MergeExclude Include="$(OutputPath)System.IO.Compression.dll"/>
<MergeExclude Include="$(OutputPath)System.Net.Http.dll"/>
<MergeExclude Include="$(OutputPath)System.Net.Http.WebRequest.dll"/>
<MergeExclude Include="$(OutputPath)System.Numerics.Vectors.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.Serialization.dll"/>
<MergeExclude Include="$(OutputPath)System.Runtime.CompilerServices.Unsafe.dll"/>
<MergeExclude Include="$(OutputPath)System.Security.dll"/>
<MergeExclude Include="$(OutputPath)System.ServiceModel.dll"/>
<MergeExclude Include="$(OutputPath)System.Xml.dll"/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@
[assembly: Guid("06662133-1292-4918-90f3-36c930c0b16f")]

[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Lucene.Net.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileProviders.Abstractions.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileSystemGlobbing.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.Primitives.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Newtonsoft.Json.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\NuGet.Commands.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\NuGet.Common.dll")]
Expand Down
3 changes: 3 additions & 0 deletions src/NuGet.Clients/NuGet.Tools/Properties/AssemblyInfo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@
[assembly: ComVisible(false)]

[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Lucene.Net.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileProviders.Abstractions.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileSystemGlobbing.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.Primitives.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Web.XmlTransform.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Newtonsoft.Json.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\NuGet.Commands.dll")]
Expand Down
3 changes: 3 additions & 0 deletions src/NuGet.Clients/NuGet.VisualStudio.Client/.vsixinclude
Original file line number Diff line number Diff line change
Expand Up @@ -34,5 +34,8 @@ NuGet.Versioning.dll

# 3rd party dlls to keep
Lucene.Net.dll
Microsoft.Extensions.FileProviders.Abstractions.dll
Microsoft.Extensions.FileSystemGlobbing.dll
Microsoft.Extensions.Primitives.dll
jeffkl marked this conversation as resolved.
Show resolved Hide resolved
Microsoft.Web.XmlTransform.dll
Newtonsoft.Json.dll
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,9 @@
<Asset Type="Microsoft.VisualStudio.MefComponent" d:Source="Project" d:ProjectName="NuGet.VisualStudio.OnlineEnvironment.Client" Path="|NuGet.VisualStudio.OnlineEnvironment.Client|" />

<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Lucene.Net.dll" AssemblyName="Lucene.Net, Version=3.0.3.0, Culture=neutral, PublicKeyToken=85089178b9ac3181" />
<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Microsoft.Extensions.FileProviders.Abstractions.dll" AssemblyName="Microsoft.Extensions.FileProviders.Abstractions, Version=5.0.0.0, Culture=neutral, PublicKeyToken=adb9793829ddae60" />
<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Microsoft.Extensions.FileSystemGlobbing.dll" AssemblyName="Microsoft.Extensions.FileSystemGlobbing, Version=5.0.0.0, Culture=neutral, PublicKeyToken=adb9793829ddae60" />
<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Microsoft.Extensions.Primitives.dll" AssemblyName="Microsoft.Extensions.Primitives, Version=5.0.0.0, Culture=neutral, PublicKeyToken=adb9793829ddae60" />
<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Microsoft.Web.XmlTransform.dll" AssemblyName="Microsoft.Web.XmlTransform, Version=2.1.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" />
<Asset Type="Microsoft.VisualStudio.Assembly" d:Source="File" Path="Newtonsoft.Json.dll" AssemblyName="Newtonsoft.Json, Version=9.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed" />
</Assets>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@
[assembly: ComVisible(false)]

[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Lucene.Net.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileProviders.Abstractions.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.FileSystemGlobbing.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Extensions.Primitives.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Microsoft.Web.XmlTransform.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\Newtonsoft.Json.dll")]
[assembly: ProvideCodeBase(CodeBase = @"$PackageFolder$\NuGet.Commands.dll")]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@
/out:$(PathToMergedNuGetPack) ^
$(ILMergePlatformArg) ^
/lib:$(OutputPath)publish ^
/allowDup ^
/internalize ^
/xmldocs ^
/log:$(ILMergeResultDir)IlMergeLog.txt</IlmergeCommand>
Expand Down Expand Up @@ -153,6 +154,7 @@
/out:$(PathToMergedNuGetPackResource) ^
$(ILMergePlatformArg) ^
/log ^
/allowDup ^
/lib:$(OutputPath)\publish ^
/internalize ^
/xmldocs ^
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
<ItemGroup>
<PackageReference Include="Microsoft.Extensions.CommandLineUtils.Sources" PrivateAssets="All" />
<PackageReference Include="System.Diagnostics.Debug" />
<PackageReference Include="System.Runtime.Serialization.Primitives" />
</ItemGroup>

<!-- Microsoft.Build.Locator is needed when debugging, but should not be used in the assemblies we insert.
Expand Down
6 changes: 3 additions & 3 deletions src/NuGet.Core/NuGet.Commands/NuGet.Commands.csproj
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project>
<Project>
<Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), 'NuGet.sln'))\build\common.props" />
<Import Project="Sdk.props" Sdk="Microsoft.NET.Sdk" />

Expand All @@ -22,8 +22,8 @@
</PropertyGroup>

<ItemGroup>
<Compile Remove="..\..\..\submodules\FileSystem\src\**\AssemblyInfo.cs" />
<Compile Include="..\..\..\submodules\FileSystem\src\Microsoft.Extensions.FileSystemGlobbing\**\*.cs;..\..\..\submodules\FileSystem\src\Microsoft.AspNet.FileProviders.Abstractions\**\*.cs;..\..\..\submodules\FileSystem\src\Microsoft.AspNet.FileProviders.Sources\**\*.cs;..\..\..\submodules\Common\src\Microsoft.Extensions.Primitives\IChangeToken.cs" Exclude="..\..\..\submodules\FileSystem\src\**\AssemblyInfo.cs;bin\**;obj\**;**\*.xproj;packages\**" />
<PackageReference Include="Microsoft.Extensions.FileSystemGlobbing" />
<PackageReference Include="Microsoft.Extensions.FileProviders.Abstractions" />
</ItemGroup>

<ItemGroup>
Expand Down
Loading