From 0ea4e6c63a02290d21b670f1f4038b3948322ddd Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Tue, 6 Feb 2024 16:23:47 -0600 Subject: [PATCH 01/40] Isolate package flow between repos --- src/SourceBuild/content/Directory.Build.props | 2 +- .../repo-projects/Directory.Build.props | 1 + .../repo-projects/Directory.Build.targets | 46 +++++++++++++++---- .../content/repo-projects/aspnetcore.proj | 9 ++-- .../content/repo-projects/cecil.proj | 1 + .../repo-projects/command-line-api.proj | 1 + .../repo-projects/deployment-tools.proj | 3 +- .../content/repo-projects/diagnostics.proj | 1 + .../content/repo-projects/dotnet.proj | 37 ++------------- .../content/repo-projects/emsdk.proj | 1 + .../content/repo-projects/format.proj | 6 +++ .../content/repo-projects/fsharp.proj | 6 ++- .../repo-projects/installer-deps.props | 24 ++++++++++ .../content/repo-projects/installer.proj | 18 ++------ .../content/repo-projects/msbuild.proj | 3 +- .../content/repo-projects/nuget-client.proj | 2 +- .../repo-projects/package-source-build.proj | 34 +++++++++++--- .../content/repo-projects/razor.proj | 3 +- .../repo-projects/roslyn-analyzers.proj | 4 +- .../content/repo-projects/roslyn.proj | 3 +- .../content/repo-projects/runtime.proj | 3 +- .../content/repo-projects/scenario-tests.proj | 3 +- .../content/repo-projects/sdk.proj | 21 ++++++--- .../repo-projects/source-build-externals.proj | 5 ++ .../content/repo-projects/sourcelink.proj | 1 + .../content/repo-projects/symreader.proj | 1 + .../content/repo-projects/templating.proj | 1 + .../content/repo-projects/test-templates.proj | 2 +- .../content/repo-projects/vstest.proj | 1 + 29 files changed, 156 insertions(+), 87 deletions(-) create mode 100644 src/SourceBuild/content/repo-projects/installer-deps.props diff --git a/src/SourceBuild/content/Directory.Build.props b/src/SourceBuild/content/Directory.Build.props index 2ab3da9ec3c9..6d2e6bcc9ec3 100644 --- a/src/SourceBuild/content/Directory.Build.props +++ b/src/SourceBuild/content/Directory.Build.props @@ -158,7 +158,7 @@ $([MSBuild]::NormalizeDirectory('$(ArtifactsObjDir)', '$(TargetArchitecture)', '$(Configuration)')) $([MSBuild]::NormalizeDirectory('$(SharedIntermediateOutputPath)', 'blob-feed')) - $([MSBuild]::NormalizeDirectory('$(SourceBuiltBlobFeedDir)', 'packages')) + $([MSBuild]::NormalizeDirectory('$(SourceBuiltBlobFeedDir)', 'packages')) $([MSBuild]::NormalizeDirectory('$(SourceBuiltBlobFeedDir)', 'assets')) $([MSBuild]::NormalizeDirectory('$(PrereqsPackagesDir)', 'prebuilt')) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.props b/src/SourceBuild/content/repo-projects/Directory.Build.props index 7c9ac1dd64ba..e0cab1d7a19e 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.props +++ b/src/SourceBuild/content/repo-projects/Directory.Build.props @@ -33,6 +33,7 @@ $(BaseIntermediateOutputPath)$([System.IO.Path]::GetFileName('$(OriginalNuGetConfigFile)')) $([MSBuild]::NormalizeDirectory('$(ArtifactsDir)', 'source-built-sdks')) + $(SourceBuiltBasePackagesPath)$(RepositoryName)/ $([MSBuild]::ValueOrDefault('$(ARCADE_BOOTSTRAP_VERSION)', '$(ArcadeSdkVersion)')) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index a06ced3c0801..3455279d8c0a 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -54,6 +54,17 @@ + + + + + source-built-%(Identity) + $(SourceBuiltBasePackagesPath)%(Identity)/ + + + + + - source-built ExtraSources - $(SourceBuiltNuGetSourceName) - $(SourceBuildSources);$(ExtraSourcesNuGetSourceName) prebuilt previously-source-built reference-packages - $(SourceBuildSources);$(PrebuiltNuGetSourceName);$(PreviouslySourceBuiltNuGetSourceName);$(ReferencePackagesNuGetSourceName) + + <_CommonSourceBuildSources Include="@(DependentRepoSourceName)" /> + <_CommonSourceBuildSources Include="$(ExtraSourcesNuGetSourceName)" Condition="'$(ExtraRestoreSourcePath)' != ''" /> + + + + + + + - <_CurrentSourceBuiltPackages Include="$(SourceBuiltPackagesPath)*.nupkg" - Exclude="$(SourceBuiltPackagesPath)*.symbols.nupkg" /> + + <_CurrentSourceBuiltPackages Include="@(DependentRepoPackageFile)" + Condition="'%(Extension)' == '.nupkg' and !$([System.String]::new('%(Filename)').EndsWith('.symbols')) " /> <_PreviouslyBuiltSourceBuiltPackages Include="$(PrebuiltSourceBuiltPackagesPath)*.nupkg" /> diff --git a/src/SourceBuild/content/repo-projects/aspnetcore.proj b/src/SourceBuild/content/repo-projects/aspnetcore.proj index be3d4294cead..c0b219d8a6c7 100644 --- a/src/SourceBuild/content/repo-projects/aspnetcore.proj +++ b/src/SourceBuild/content/repo-projects/aspnetcore.proj @@ -23,11 +23,12 @@ - - - - + + + + + diff --git a/src/SourceBuild/content/repo-projects/cecil.proj b/src/SourceBuild/content/repo-projects/cecil.proj index c4068fe7b99c..563e7998b4b3 100644 --- a/src/SourceBuild/content/repo-projects/cecil.proj +++ b/src/SourceBuild/content/repo-projects/cecil.proj @@ -2,6 +2,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/command-line-api.proj b/src/SourceBuild/content/repo-projects/command-line-api.proj index a16b2412a19b..198cebc2b07d 100644 --- a/src/SourceBuild/content/repo-projects/command-line-api.proj +++ b/src/SourceBuild/content/repo-projects/command-line-api.proj @@ -8,6 +8,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/deployment-tools.proj b/src/SourceBuild/content/repo-projects/deployment-tools.proj index 8ef3a8a5a61f..e25768a77c40 100644 --- a/src/SourceBuild/content/repo-projects/deployment-tools.proj +++ b/src/SourceBuild/content/repo-projects/deployment-tools.proj @@ -2,7 +2,8 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/diagnostics.proj b/src/SourceBuild/content/repo-projects/diagnostics.proj index c4068fe7b99c..563e7998b4b3 100644 --- a/src/SourceBuild/content/repo-projects/diagnostics.proj +++ b/src/SourceBuild/content/repo-projects/diagnostics.proj @@ -2,6 +2,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/dotnet.proj b/src/SourceBuild/content/repo-projects/dotnet.proj index 0de74d67944c..4137fd8ca6e0 100644 --- a/src/SourceBuild/content/repo-projects/dotnet.proj +++ b/src/SourceBuild/content/repo-projects/dotnet.proj @@ -10,43 +10,12 @@ - We need an "empty" build after the last real repo for prebuilt detection to work - this is that file. - If we have a repo that is not in sdk's dependency tree, we can still build it by including it here. --> - - - - + + - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - diff --git a/src/SourceBuild/content/repo-projects/emsdk.proj b/src/SourceBuild/content/repo-projects/emsdk.proj index df9c063263e6..65f811a0bcac 100644 --- a/src/SourceBuild/content/repo-projects/emsdk.proj +++ b/src/SourceBuild/content/repo-projects/emsdk.proj @@ -12,6 +12,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/format.proj b/src/SourceBuild/content/repo-projects/format.proj index 9418953b7e8d..12e84f7a56a0 100644 --- a/src/SourceBuild/content/repo-projects/format.proj +++ b/src/SourceBuild/content/repo-projects/format.proj @@ -2,9 +2,15 @@ + + + + + + diff --git a/src/SourceBuild/content/repo-projects/fsharp.proj b/src/SourceBuild/content/repo-projects/fsharp.proj index bac7243b5321..cc0bfde7289d 100644 --- a/src/SourceBuild/content/repo-projects/fsharp.proj +++ b/src/SourceBuild/content/repo-projects/fsharp.proj @@ -20,8 +20,10 @@ - - + + + + diff --git a/src/SourceBuild/content/repo-projects/installer-deps.props b/src/SourceBuild/content/repo-projects/installer-deps.props new file mode 100644 index 000000000000..2dbf035e6b47 --- /dev/null +++ b/src/SourceBuild/content/repo-projects/installer-deps.props @@ -0,0 +1,24 @@ + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/src/SourceBuild/content/repo-projects/installer.proj b/src/SourceBuild/content/repo-projects/installer.proj index bcc0fe5f7534..bc65393006e9 100644 --- a/src/SourceBuild/content/repo-projects/installer.proj +++ b/src/SourceBuild/content/repo-projects/installer.proj @@ -10,6 +10,8 @@ $(FlagParameterPrefix)pack $(FlagParameterPrefix)publish + + true $(BuildArgs) $(FlagParameterPrefix)runtime-id $(TargetRid) @@ -34,20 +36,8 @@ $(BuildArgs) /p:UsePortableLinuxSharedFramework=false - - - - - - - - - - - - - - + + - - - $(SourceBuiltPackagesPath)SourceBuildReferencePackages/ + SourceBuildReferencePackages + $(SourceBuiltPackagesPath)$(SourceBuildReferencePackagesDestinationDirName)/ @@ -47,10 +47,32 @@ Overwrite="true" /> - + + + <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(SourceBuiltBasePackagesPath)))" + Exclude="$(SourceBuiltBasePackagesPath)package-source-build" /> + + + + + <_SourceBuiltLayoutDir>$(BaseIntermediateOutputPath)artifacts-layout/ + + + + + + + + + + + + diff --git a/src/SourceBuild/content/repo-projects/razor.proj b/src/SourceBuild/content/repo-projects/razor.proj index 800961adeb73..d95de454c619 100644 --- a/src/SourceBuild/content/repo-projects/razor.proj +++ b/src/SourceBuild/content/repo-projects/razor.proj @@ -2,8 +2,9 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj b/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj index 23819f4994d4..aa624342037a 100644 --- a/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj +++ b/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj @@ -8,8 +8,8 @@ - - + + diff --git a/src/SourceBuild/content/repo-projects/roslyn.proj b/src/SourceBuild/content/repo-projects/roslyn.proj index 68d4995140eb..a1c47c04c70d 100644 --- a/src/SourceBuild/content/repo-projects/roslyn.proj +++ b/src/SourceBuild/content/repo-projects/roslyn.proj @@ -30,8 +30,9 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/runtime.proj b/src/SourceBuild/content/repo-projects/runtime.proj index 35bbd9b03822..a056276a7f29 100644 --- a/src/SourceBuild/content/repo-projects/runtime.proj +++ b/src/SourceBuild/content/repo-projects/runtime.proj @@ -31,8 +31,9 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/scenario-tests.proj b/src/SourceBuild/content/repo-projects/scenario-tests.proj index 25c6f1b9cd57..b71f89c20748 100644 --- a/src/SourceBuild/content/repo-projects/scenario-tests.proj +++ b/src/SourceBuild/content/repo-projects/scenario-tests.proj @@ -1,9 +1,10 @@ + + - diff --git a/src/SourceBuild/content/repo-projects/sdk.proj b/src/SourceBuild/content/repo-projects/sdk.proj index 69c6a2826dc4..b22bcb759378 100644 --- a/src/SourceBuild/content/repo-projects/sdk.proj +++ b/src/SourceBuild/content/repo-projects/sdk.proj @@ -14,17 +14,26 @@ - + + + + + + - + + + + + + + + - - - - + diff --git a/src/SourceBuild/content/repo-projects/source-build-externals.proj b/src/SourceBuild/content/repo-projects/source-build-externals.proj index a391c4cd6689..c86415ade2f4 100644 --- a/src/SourceBuild/content/repo-projects/source-build-externals.proj +++ b/src/SourceBuild/content/repo-projects/source-build-externals.proj @@ -10,4 +10,9 @@ true + + + + + diff --git a/src/SourceBuild/content/repo-projects/sourcelink.proj b/src/SourceBuild/content/repo-projects/sourcelink.proj index 87026a8e9930..1eb75e01a1be 100644 --- a/src/SourceBuild/content/repo-projects/sourcelink.proj +++ b/src/SourceBuild/content/repo-projects/sourcelink.proj @@ -10,6 +10,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/symreader.proj b/src/SourceBuild/content/repo-projects/symreader.proj index c4068fe7b99c..563e7998b4b3 100644 --- a/src/SourceBuild/content/repo-projects/symreader.proj +++ b/src/SourceBuild/content/repo-projects/symreader.proj @@ -2,6 +2,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/templating.proj b/src/SourceBuild/content/repo-projects/templating.proj index 999d5a167488..6c108ed896b9 100644 --- a/src/SourceBuild/content/repo-projects/templating.proj +++ b/src/SourceBuild/content/repo-projects/templating.proj @@ -11,6 +11,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/test-templates.proj b/src/SourceBuild/content/repo-projects/test-templates.proj index a272afd4f160..f3957fa25f92 100644 --- a/src/SourceBuild/content/repo-projects/test-templates.proj +++ b/src/SourceBuild/content/repo-projects/test-templates.proj @@ -6,7 +6,7 @@ - + diff --git a/src/SourceBuild/content/repo-projects/vstest.proj b/src/SourceBuild/content/repo-projects/vstest.proj index c228f58a635b..39891d188188 100644 --- a/src/SourceBuild/content/repo-projects/vstest.proj +++ b/src/SourceBuild/content/repo-projects/vstest.proj @@ -11,6 +11,7 @@ + From 3878626094371851cb220b836c647a77c3e32e9f Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Mon, 12 Feb 2024 14:35:27 -0600 Subject: [PATCH 02/40] Compute transitive dependencies --- .../repo-projects/Directory.Build.targets | 30 ++++--------------- .../content/repo-projects/aspnetcore.proj | 2 -- .../content/repo-projects/dotnet.proj | 3 -- .../content/repo-projects/format.proj | 4 --- .../content/repo-projects/fsharp.proj | 2 -- .../content/repo-projects/installer.proj | 22 ++++++++++++-- .../content/repo-projects/msbuild.proj | 2 -- .../repo-projects/package-source-build.proj | 2 -- .../content/repo-projects/razor.proj | 2 -- .../content/repo-projects/roslyn.proj | 2 -- .../content/repo-projects/runtime.proj | 1 - .../content/repo-projects/scenario-tests.proj | 1 - .../content/repo-projects/sdk.proj | 10 ------- .../content/repo-projects/templating.proj | 1 - .../content/repo-projects/vstest.proj | 1 - .../content/repo-projects/winforms.proj | 1 - .../content/repo-projects/wpf.proj | 1 - 17 files changed, 26 insertions(+), 61 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 3455279d8c0a..cd93c61f5cb1 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -25,26 +25,6 @@ $([MSBuild]::NormalizePath('$(ProjectDirectory)', 'artifacts', 'RepoManifest.xml')) - - - - - - - - - - - - - - - - - - - - @@ -55,9 +35,10 @@ - + - + source-built-%(Identity) $(SourceBuiltBasePackagesPath)%(Identity)/ @@ -286,11 +267,12 @@ - + - <_DependentProject Include="@(RepositoryReference -> '%(Identity).proj')" /> + <_DependentProject Include="@(TransitiveRepositoryReference -> '%(Identity).proj')" /> - - diff --git a/src/SourceBuild/content/repo-projects/dotnet.proj b/src/SourceBuild/content/repo-projects/dotnet.proj index 4137fd8ca6e0..94891c25ce0e 100644 --- a/src/SourceBuild/content/repo-projects/dotnet.proj +++ b/src/SourceBuild/content/repo-projects/dotnet.proj @@ -10,9 +10,6 @@ - We need an "empty" build after the last real repo for prebuilt detection to work - this is that file. - If we have a repo that is not in sdk's dependency tree, we can still build it by including it here. --> - - - diff --git a/src/SourceBuild/content/repo-projects/format.proj b/src/SourceBuild/content/repo-projects/format.proj index 12e84f7a56a0..efffabcb3b02 100644 --- a/src/SourceBuild/content/repo-projects/format.proj +++ b/src/SourceBuild/content/repo-projects/format.proj @@ -1,12 +1,8 @@ - - - - diff --git a/src/SourceBuild/content/repo-projects/fsharp.proj b/src/SourceBuild/content/repo-projects/fsharp.proj index cc0bfde7289d..dd18fea7fc8a 100644 --- a/src/SourceBuild/content/repo-projects/fsharp.proj +++ b/src/SourceBuild/content/repo-projects/fsharp.proj @@ -19,9 +19,7 @@ - - diff --git a/src/SourceBuild/content/repo-projects/installer.proj b/src/SourceBuild/content/repo-projects/installer.proj index bc65393006e9..45075464d4bf 100644 --- a/src/SourceBuild/content/repo-projects/installer.proj +++ b/src/SourceBuild/content/repo-projects/installer.proj @@ -36,8 +36,26 @@ $(BuildArgs) /p:UsePortableLinuxSharedFramework=false - - + + + + + + + + + + + + + + + + + + + + + + + <_TransitiveRepositoryReference Include="@(RepositoryReference)" /> + + + + + + + + + <_TransitiveRepositoryReference Include="@(_DependencyTransitiveRepositoryReference)" + RemoveMetadata="MSBuildSourceProjectFile;MSBuildSourceTargetName;OriginalItemSpec" /> + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + <_NextIndent>$(DependencyGraphIndent)__ + + + + + + + + + + + <_DependencyGraphString>@(_DependencyGraphString, '') + <_LineBreak>%0a + $(DependencyGraphIndent)-_$(RepositoryName)$(_LineBreak)$(_DependencyGraphString) + + From edebea74ce5d93ed82140d750bc729322b03d233 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Mon, 12 Feb 2024 14:54:58 -0600 Subject: [PATCH 04/40] Fix dependency on scenario-tests --- src/SourceBuild/content/repo-projects/dotnet.proj | 1 + src/SourceBuild/content/repo-projects/installer.proj | 1 - src/SourceBuild/content/repo-projects/package-source-build.proj | 1 + 3 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/dotnet.proj b/src/SourceBuild/content/repo-projects/dotnet.proj index 94891c25ce0e..732eb34f9932 100644 --- a/src/SourceBuild/content/repo-projects/dotnet.proj +++ b/src/SourceBuild/content/repo-projects/dotnet.proj @@ -13,6 +13,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/installer.proj b/src/SourceBuild/content/repo-projects/installer.proj index 45075464d4bf..d1b3dacf3c71 100644 --- a/src/SourceBuild/content/repo-projects/installer.proj +++ b/src/SourceBuild/content/repo-projects/installer.proj @@ -48,7 +48,6 @@ - diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 69a9c0630504..a329557630dc 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -71,6 +71,7 @@ + From 5b8a5a232b1a6d26636cbe504abaeb45a8dca7c9 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Mon, 12 Feb 2024 14:55:31 -0600 Subject: [PATCH 05/40] Remove duplicate installer dependency --- .../content/repo-projects/package-source-build.proj | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index a329557630dc..c54e6b327906 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -7,10 +7,6 @@ false - - - - From 566f613417d62293ab682115143de735661e9648 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Mon, 12 Feb 2024 15:49:02 -0600 Subject: [PATCH 06/40] Delete obsolete installer-deps --- .../repo-projects/installer-deps.props | 24 ------------------- 1 file changed, 24 deletions(-) delete mode 100644 src/SourceBuild/content/repo-projects/installer-deps.props diff --git a/src/SourceBuild/content/repo-projects/installer-deps.props b/src/SourceBuild/content/repo-projects/installer-deps.props deleted file mode 100644 index 2dbf035e6b47..000000000000 --- a/src/SourceBuild/content/repo-projects/installer-deps.props +++ /dev/null @@ -1,24 +0,0 @@ - - - - - - - - - - - - - - - - - - - - - - - - From 8ce569e8889352e040203353af4db43a9f457e5b Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Tue, 13 Feb 2024 12:38:50 -0600 Subject: [PATCH 07/40] Add conditional deps specific to source build --- src/SourceBuild/content/repo-projects/aspnetcore.proj | 2 +- src/SourceBuild/content/repo-projects/format.proj | 5 ++--- src/SourceBuild/content/repo-projects/fsharp.proj | 1 - src/SourceBuild/content/repo-projects/nuget-client.proj | 4 ++-- src/SourceBuild/content/repo-projects/razor.proj | 3 ++- src/SourceBuild/content/repo-projects/roslyn.proj | 3 ++- src/SourceBuild/content/repo-projects/sdk.proj | 2 ++ src/SourceBuild/content/repo-projects/sourcelink.proj | 1 - src/SourceBuild/content/repo-projects/templating.proj | 2 +- src/SourceBuild/content/repo-projects/vstest.proj | 2 +- 10 files changed, 13 insertions(+), 12 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/aspnetcore.proj b/src/SourceBuild/content/repo-projects/aspnetcore.proj index 1dae63bfe628..fe20eb430019 100644 --- a/src/SourceBuild/content/repo-projects/aspnetcore.proj +++ b/src/SourceBuild/content/repo-projects/aspnetcore.proj @@ -25,7 +25,7 @@ - + diff --git a/src/SourceBuild/content/repo-projects/format.proj b/src/SourceBuild/content/repo-projects/format.proj index efffabcb3b02..c1aed107e367 100644 --- a/src/SourceBuild/content/repo-projects/format.proj +++ b/src/SourceBuild/content/repo-projects/format.proj @@ -1,11 +1,10 @@ - - + + - diff --git a/src/SourceBuild/content/repo-projects/fsharp.proj b/src/SourceBuild/content/repo-projects/fsharp.proj index dd18fea7fc8a..a38e59e1a3ab 100644 --- a/src/SourceBuild/content/repo-projects/fsharp.proj +++ b/src/SourceBuild/content/repo-projects/fsharp.proj @@ -21,7 +21,6 @@ - diff --git a/src/SourceBuild/content/repo-projects/nuget-client.proj b/src/SourceBuild/content/repo-projects/nuget-client.proj index 37ab58a63d6f..896dc622e871 100644 --- a/src/SourceBuild/content/repo-projects/nuget-client.proj +++ b/src/SourceBuild/content/repo-projects/nuget-client.proj @@ -9,9 +9,9 @@ - + - + diff --git a/src/SourceBuild/content/repo-projects/razor.proj b/src/SourceBuild/content/repo-projects/razor.proj index 74ce7402d5e1..7dd9f0435467 100644 --- a/src/SourceBuild/content/repo-projects/razor.proj +++ b/src/SourceBuild/content/repo-projects/razor.proj @@ -1,7 +1,8 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/roslyn.proj b/src/SourceBuild/content/repo-projects/roslyn.proj index 34ad46b9e9d4..4917e12dffe2 100644 --- a/src/SourceBuild/content/repo-projects/roslyn.proj +++ b/src/SourceBuild/content/repo-projects/roslyn.proj @@ -28,7 +28,8 @@ - + + diff --git a/src/SourceBuild/content/repo-projects/sdk.proj b/src/SourceBuild/content/repo-projects/sdk.proj index 4dc18056efb8..9983f62d6086 100644 --- a/src/SourceBuild/content/repo-projects/sdk.proj +++ b/src/SourceBuild/content/repo-projects/sdk.proj @@ -17,9 +17,11 @@ + + diff --git a/src/SourceBuild/content/repo-projects/sourcelink.proj b/src/SourceBuild/content/repo-projects/sourcelink.proj index 1eb75e01a1be..2efaedbbcb37 100644 --- a/src/SourceBuild/content/repo-projects/sourcelink.proj +++ b/src/SourceBuild/content/repo-projects/sourcelink.proj @@ -8,7 +8,6 @@ - diff --git a/src/SourceBuild/content/repo-projects/templating.proj b/src/SourceBuild/content/repo-projects/templating.proj index 8b1d44af6ca1..a3fa0bcbd112 100644 --- a/src/SourceBuild/content/repo-projects/templating.proj +++ b/src/SourceBuild/content/repo-projects/templating.proj @@ -8,7 +8,7 @@ - + diff --git a/src/SourceBuild/content/repo-projects/vstest.proj b/src/SourceBuild/content/repo-projects/vstest.proj index 6495f20b7290..e1c80094317f 100644 --- a/src/SourceBuild/content/repo-projects/vstest.proj +++ b/src/SourceBuild/content/repo-projects/vstest.proj @@ -8,7 +8,7 @@ - + From d18498a7931f3306aa10ee2c08b1edbdfb74308f Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Fri, 16 Feb 2024 15:35:36 -0600 Subject: [PATCH 08/40] Updates to integrate with artifact path refactoring --- .../repo-projects/Directory.Build.props | 6 +- .../repo-projects/Directory.Build.targets | 72 +++++++++---------- .../content/repo-projects/aspnetcore.proj | 1 + .../repo-projects/package-source-build.proj | 48 ++----------- .../repo-projects/roslyn-analyzers.proj | 1 + 5 files changed, 50 insertions(+), 78 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.props b/src/SourceBuild/content/repo-projects/Directory.Build.props index a216ef3de041..ec56c01a858d 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.props +++ b/src/SourceBuild/content/repo-projects/Directory.Build.props @@ -33,7 +33,11 @@ $(BaseIntermediateOutputPath)$([System.IO.Path]::GetFileName('$(OriginalNuGetConfigFile)')) $([MSBuild]::NormalizeDirectory('$(ArtifactsDir)', 'source-built-sdks')) - $(SourceBuiltBasePackagesPath)$(RepositoryName)/ + $([MSBuild]::NormalizeDirectory('$(ArtifactsShippingPackagesDir)', '$(RepositoryName)')) + $(ReferencePackagesDir) + $([MSBuild]::NormalizeDirectory('$(ArtifactsNonShippingPackagesDir)', '$(RepositoryName)')) + $(ReferencePackagesDir) + $([MSBuild]::ValueOrDefault('$(ARCADE_BOOTSTRAP_VERSION)', '$(ArcadeSdkVersion)')) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 6b99bdc8f431..7a4457071341 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -41,15 +41,24 @@ + - - - source-built-%(Identity) - $(SourceBuiltBasePackagesPath)%(Identity)/ + + + + source-built-%(Identity) + source-built-transport-%(Identity) + $(ArtifactsShippingPackagesDir)/%(Identity)/ + $(ArtifactsNonShippingPackagesDir)/%(Identity)/ - - + + + + + + @@ -73,8 +82,6 @@ Inputs="$(MSBuildProjectFullPath)" Outputs="$(BaseIntermediateOutputPath)UpdateNuGetConfig.complete"> - - source-built-transport ExtraSources @@ -87,7 +94,6 @@ <_CommonSourceBuildSources Include="@(DependentRepoSourceName)" /> - <_CommonSourceBuildSources Include="$(SourceBuiltNonShippingNuGetSourceName)" /> <_CommonSourceBuildSources Include="$(ExtraSourcesNuGetSourceName)" Condition="'$(ExtraRestoreSourcePath)' != ''" /> @@ -136,13 +142,14 @@ Condition="'$(DotNetBuildSourceOnly)' == 'true'" /> + SourceName="%(RepositoryReferenceInfo.NonShippingSourceName)" + SourcePath="%(RepositoryReferenceInfo.NonShippingPackagesPath)" + Condition=" '@(RepositoryReferenceInfo)' != '' " /> - <_CurrentSourceBuiltPackages Include="@(DependentRepoPackageFile)" Condition="!$([System.String]::Copy('%(Identity)').EndsWith('.symbols.nupkg'))" /> @@ -339,8 +345,8 @@ @@ -412,14 +418,12 @@ - $(ArtifactsNonShippingPackagesDir) - $(ReferencePackagesDir) + $(RepoArtifactsNonShippingPackagesDir) - $(ArtifactsShippingPackagesDir) - $(ReferencePackagesDir) + $(RepoArtifactsShippingPackagesDir) @@ -520,8 +524,7 @@ Inputs="$(MSBuildProjectFullPath)" Outputs="$(BaseIntermediateOutputPath)ExtractToolPackage.complete"> - <_ToolPackagesRoot Condition="'$(RepositoryName)' != 'source-build-reference-packages'">$(ArtifactsNonShippingPackagesDir) - <_ToolPackagesRoot Condition="'$(RepositoryName)' == 'source-build-reference-packages'">$(ReferencePackagesDir) + <_ToolPackagesRoot>$(RepoArtifactsNonShippingPackagesDir) @@ -731,20 +734,17 @@ - - - - - - - - - - - - - - + + + + + + + + + + + diff --git a/src/SourceBuild/content/repo-projects/aspnetcore.proj b/src/SourceBuild/content/repo-projects/aspnetcore.proj index 5e576dabda60..e0dca23c0354 100644 --- a/src/SourceBuild/content/repo-projects/aspnetcore.proj +++ b/src/SourceBuild/content/repo-projects/aspnetcore.proj @@ -21,6 +21,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 3487b6d02fd0..9da0a9a576b5 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -10,79 +10,45 @@ - - $([MSBuild]::NormalizeDirectory('$(BaseIntermediateOutputPath)', '$(SourceBuiltArtifactsTarballName)')) - $([MSBuild]::NormalizeDirectory('$(SourceBuildTarballStagingDir)', 'SourceBuildReferencePackages')) - - - - - - - - - - - - - - - - - - - - - $(ArtifactsAssetsDir)$(SourceBuiltArtifactsTarballName).$(SourceBuiltSdkVersion).$(TargetRid)$(ArchiveExtension) .version - + - <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(SourceBuiltBasePackagesPath)))" - Exclude="$(SourceBuiltBasePackagesPath)package-source-build" /> + <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(ArtifactsShippingPackagesDir)))" /> + <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(ArtifactsNonShippingPackagesDir)))" /> <_SourceBuiltLayoutDir>$(BaseIntermediateOutputPath)artifacts-layout/ + SourceBuildReferencePackages - + - + - - diff --git a/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj b/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj index aa624342037a..4de42e5956fc 100644 --- a/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj +++ b/src/SourceBuild/content/repo-projects/roslyn-analyzers.proj @@ -8,6 +8,7 @@ + From 608e3adab65aca27adcc828a17383c6b401466dc Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:33:52 -0600 Subject: [PATCH 09/40] Add back primary dependencies --- src/SourceBuild/content/repo-projects/aspnetcore.proj | 4 +++- src/SourceBuild/content/repo-projects/format.proj | 4 ++++ src/SourceBuild/content/repo-projects/fsharp.proj | 2 ++ src/SourceBuild/content/repo-projects/msbuild.proj | 2 ++ src/SourceBuild/content/repo-projects/runtime.proj | 1 + src/SourceBuild/content/repo-projects/scenario-tests.proj | 1 + src/SourceBuild/content/repo-projects/sdk.proj | 8 ++++++++ src/SourceBuild/content/repo-projects/sourcelink.proj | 1 + src/SourceBuild/content/repo-projects/vstest.proj | 1 + src/SourceBuild/content/repo-projects/winforms.proj | 1 + src/SourceBuild/content/repo-projects/wpf.proj | 1 + 11 files changed, 25 insertions(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/aspnetcore.proj b/src/SourceBuild/content/repo-projects/aspnetcore.proj index e0dca23c0354..d5a6f748e10f 100644 --- a/src/SourceBuild/content/repo-projects/aspnetcore.proj +++ b/src/SourceBuild/content/repo-projects/aspnetcore.proj @@ -21,8 +21,10 @@ - + + + diff --git a/src/SourceBuild/content/repo-projects/format.proj b/src/SourceBuild/content/repo-projects/format.proj index c1aed107e367..f311513cafe1 100644 --- a/src/SourceBuild/content/repo-projects/format.proj +++ b/src/SourceBuild/content/repo-projects/format.proj @@ -1,8 +1,12 @@ + + + + diff --git a/src/SourceBuild/content/repo-projects/fsharp.proj b/src/SourceBuild/content/repo-projects/fsharp.proj index a38e59e1a3ab..52e43fbb828c 100644 --- a/src/SourceBuild/content/repo-projects/fsharp.proj +++ b/src/SourceBuild/content/repo-projects/fsharp.proj @@ -19,7 +19,9 @@ + + diff --git a/src/SourceBuild/content/repo-projects/msbuild.proj b/src/SourceBuild/content/repo-projects/msbuild.proj index 5d62b7b39af2..e4259a9038e0 100644 --- a/src/SourceBuild/content/repo-projects/msbuild.proj +++ b/src/SourceBuild/content/repo-projects/msbuild.proj @@ -8,7 +8,9 @@ + + diff --git a/src/SourceBuild/content/repo-projects/runtime.proj b/src/SourceBuild/content/repo-projects/runtime.proj index 6991872c92cb..c911d46aa5e8 100644 --- a/src/SourceBuild/content/repo-projects/runtime.proj +++ b/src/SourceBuild/content/repo-projects/runtime.proj @@ -31,6 +31,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/scenario-tests.proj b/src/SourceBuild/content/repo-projects/scenario-tests.proj index 2a163ae46cb7..b71f89c20748 100644 --- a/src/SourceBuild/content/repo-projects/scenario-tests.proj +++ b/src/SourceBuild/content/repo-projects/scenario-tests.proj @@ -1,6 +1,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/sdk.proj b/src/SourceBuild/content/repo-projects/sdk.proj index 9983f62d6086..b22bcb759378 100644 --- a/src/SourceBuild/content/repo-projects/sdk.proj +++ b/src/SourceBuild/content/repo-projects/sdk.proj @@ -13,15 +13,23 @@ + + + + + + + + diff --git a/src/SourceBuild/content/repo-projects/sourcelink.proj b/src/SourceBuild/content/repo-projects/sourcelink.proj index 2efaedbbcb37..1eb75e01a1be 100644 --- a/src/SourceBuild/content/repo-projects/sourcelink.proj +++ b/src/SourceBuild/content/repo-projects/sourcelink.proj @@ -8,6 +8,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/vstest.proj b/src/SourceBuild/content/repo-projects/vstest.proj index e1c80094317f..4a7faf4e4098 100644 --- a/src/SourceBuild/content/repo-projects/vstest.proj +++ b/src/SourceBuild/content/repo-projects/vstest.proj @@ -7,6 +7,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/winforms.proj b/src/SourceBuild/content/repo-projects/winforms.proj index 05371f56f51c..489f0055d7c0 100644 --- a/src/SourceBuild/content/repo-projects/winforms.proj +++ b/src/SourceBuild/content/repo-projects/winforms.proj @@ -8,6 +8,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/wpf.proj b/src/SourceBuild/content/repo-projects/wpf.proj index a84fac4f4629..e5ee42cc7bcf 100644 --- a/src/SourceBuild/content/repo-projects/wpf.proj +++ b/src/SourceBuild/content/repo-projects/wpf.proj @@ -9,6 +9,7 @@ + From a7a6f797cf0c959a333b8362958a0873b3540f5b Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:34:13 -0600 Subject: [PATCH 10/40] Move psb references --- .../content/repo-projects/package-source-build.proj | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 9da0a9a576b5..cfc3672f8f8c 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -7,6 +7,11 @@ false + + + + + @@ -51,9 +56,4 @@ - - - - - From bdaa440797890f0ef606393025d6967d656c4917 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:34:33 -0600 Subject: [PATCH 11/40] Use Exclude instead of Condition --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 7a4457071341..546ea2ec37c0 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -47,7 +47,7 @@ + Exclude="source-build-reference-packages"> source-built-%(Identity) source-built-transport-%(Identity) $(ArtifactsShippingPackagesDir)/%(Identity)/ From 241ad6fec829efd38530b3744604fae68f0f5efe Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:34:44 -0600 Subject: [PATCH 12/40] whitespace --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 1 + 1 file changed, 1 insertion(+) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 546ea2ec37c0..26503d5c6b85 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -54,6 +54,7 @@ $(ArtifactsNonShippingPackagesDir)/%(Identity)/ + From a50512dc17ae245c0ac70d263c90b7c632961889 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:35:37 -0600 Subject: [PATCH 13/40] Remove MakeDir for package dirs --- .../content/repo-projects/Directory.Build.targets | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 26503d5c6b85..76b6605ef7f7 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -145,12 +145,12 @@ + Condition="Exists('%(RepositoryReferenceInfo.ShippingPackagesPath)')" /> + Condition="Exists('%(RepositoryReferenceInfo.NonShippingPackagesPath)')" /> From c97b8d2c442032017f5ec4a57a7421841660631c Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:36:07 -0600 Subject: [PATCH 14/40] Add comment for embedding --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 76b6605ef7f7..b6e0a3ea9805 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -174,6 +174,8 @@ + From 8b3f3b271825f5c273d951a1c95160516e6ad51e Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:36:23 -0600 Subject: [PATCH 15/40] indenting --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index b6e0a3ea9805..1490ded5b2d4 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -177,7 +177,7 @@ - + - - - - - - - - - - - + + + + + + + + + + + From 0540025c557a98c47a62c7f24d8a2335c20d179a Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Wed, 21 Feb 2024 12:37:17 -0600 Subject: [PATCH 17/40] Remove SkipEnsurePackagesCreated --- src/SourceBuild/content/repo-projects/installer.proj | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/installer.proj b/src/SourceBuild/content/repo-projects/installer.proj index fb749c74b4de..17fd81a2df98 100644 --- a/src/SourceBuild/content/repo-projects/installer.proj +++ b/src/SourceBuild/content/repo-projects/installer.proj @@ -11,8 +11,6 @@ $(FlagParameterPrefix)pack $(FlagParameterPrefix)publish - true - $(BuildArgs) $(FlagParameterPrefix)runtime-id $(TargetRid) $(FlagParameterPrefix)pack $(FlagParameterPrefix)publish - + $(BuildArgs) $(FlagParameterPrefix)runtime-id $(TargetRid) - <_SourceBuiltLayoutDir>$(BaseIntermediateOutputPath)artifacts-layout/ + <_SourceBuiltLayoutDir>$([MSBuild]::NormalizeDirectory('$(BaseIntermediateOutputPath)', 'artifacts-layout')) SourceBuildReferencePackages - - Date: Thu, 22 Feb 2024 08:22:05 -0600 Subject: [PATCH 31/40] Simplify package file collection --- .../content/repo-projects/package-source-build.proj | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index fed6a42df335..77798b1b1d66 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -30,9 +30,8 @@ Overwrite="true" /> - <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(ArtifactsShippingPackagesDir)))" /> - <_AllRepoDirs Include="$([System.IO.Directory]::GetDirectories($(ArtifactsNonShippingPackagesDir)))" /> - <_AllRepoFiles Include="%(_AllRepoDirs.Identity)/**" /> + <_AllRepoFiles Include="$(ArtifactsShippingPackagesDir)/**" /> + <_AllRepoFiles Include="$(ArtifactsNonShippingPackagesDir)/**" /> <_ReferencePackageFiles Include="$(ReferencePackagesDir)**" /> From a0599e10e5c3a47a687e84668ecb239e7f27a8cc Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:24:29 -0600 Subject: [PATCH 32/40] Add whitespace --- src/SourceBuild/content/repo-projects/package-source-build.proj | 1 + 1 file changed, 1 insertion(+) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 77798b1b1d66..8baae7fc8b31 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -41,6 +41,7 @@ <_SourceBuiltLayoutDir>$([MSBuild]::NormalizeDirectory('$(BaseIntermediateOutputPath)', 'artifacts-layout')) SourceBuildReferencePackages + From 6e800d70a7451ebf467b75a28e3f34c1f151acda Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:25:15 -0600 Subject: [PATCH 33/40] Simplify _BuildSources --- .../content/repo-projects/Directory.Build.targets | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 575e85353a5f..1b6ec61a1c13 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -100,9 +100,8 @@ <_BuildSources Condition="'$(DotNetBuildSourceOnly)' == 'true'" - Include="$(PrebuiltNuGetSourceName);$(PreviouslySourceBuiltNuGetSourceName);$(ReferencePackagesNuGetSourceName);@(_CommonBuildSources)" /> - <_BuildSources Condition="'$(DotNetBuildSourceOnly)' != 'true'" - Include="@(_CommonBuildSources)" /> + Include="$(PrebuiltNuGetSourceName);$(PreviouslySourceBuiltNuGetSourceName);$(ReferencePackagesNuGetSourceName)" /> + <_BuildSources Include="@(_CommonBuildSources)" /> From 59013c60f57ca8d8278bee5167d4fdd107f06055 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:30:26 -0600 Subject: [PATCH 34/40] Get repo refs in parallel --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 1b6ec61a1c13..2677db09128c 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -718,7 +718,8 @@ + Targets="GetTransitiveRepositoryReferences" + BuildInParallel="true"> From 28f645a9fff9494de5b69df18b50434c39297c96 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:30:36 -0600 Subject: [PATCH 35/40] Use Distinct --- .../content/repo-projects/Directory.Build.targets | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 2677db09128c..df5ce45b942e 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -726,11 +726,10 @@ + reference. To prevent this from happening, use the Distinct method which operates on just the Identity to get a unique set + of values. --> - <_TransitiveRepositoryReference Include="@(_DependencyTransitiveRepositoryReference)" - RemoveMetadata="MSBuildSourceProjectFile;MSBuildSourceTargetName;OriginalItemSpec" /> - + From 0dfac72c7bc1373fa7352b8c29c230ce06b67975 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:31:01 -0600 Subject: [PATCH 36/40] whitespace --- src/SourceBuild/content/repo-projects/Directory.Build.targets | 1 + 1 file changed, 1 insertion(+) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index df5ce45b942e..bda2d117a2a4 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -784,4 +784,5 @@ $(DependencyGraphIndent)-_$(RepositoryName)$(_LineBreak)$(_DependencyGraphString) + From 2c3836bbf7060f2963ff9deb690d1804dafcc7c9 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 08:35:31 -0600 Subject: [PATCH 37/40] Move windowsdesktop dep from sdk to installer --- src/SourceBuild/content/repo-projects/installer.proj | 1 + src/SourceBuild/content/repo-projects/sdk.proj | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/src/SourceBuild/content/repo-projects/installer.proj b/src/SourceBuild/content/repo-projects/installer.proj index f38b40fc29eb..cb5fcc0512aa 100644 --- a/src/SourceBuild/content/repo-projects/installer.proj +++ b/src/SourceBuild/content/repo-projects/installer.proj @@ -51,6 +51,7 @@ + diff --git a/src/SourceBuild/content/repo-projects/sdk.proj b/src/SourceBuild/content/repo-projects/sdk.proj index 3d0fa1d44ee0..d307692ac8c1 100644 --- a/src/SourceBuild/content/repo-projects/sdk.proj +++ b/src/SourceBuild/content/repo-projects/sdk.proj @@ -30,7 +30,6 @@ - From 9a25ae6e4819d9f9d1a327871fcf01745f665501 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 09:10:06 -0600 Subject: [PATCH 38/40] Revert "Use Distinct" This reverts commit 28f645a9fff9494de5b69df18b50434c39297c96. --- .../content/repo-projects/Directory.Build.targets | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/Directory.Build.targets b/src/SourceBuild/content/repo-projects/Directory.Build.targets index 24ba3bfd91a2..bfa4fc5945dc 100644 --- a/src/SourceBuild/content/repo-projects/Directory.Build.targets +++ b/src/SourceBuild/content/repo-projects/Directory.Build.targets @@ -726,10 +726,11 @@ + reference. To prevent this from happening, we need to remove the extra metadata so the two references can be seen as duplicates. --> - + <_TransitiveRepositoryReference Include="@(_DependencyTransitiveRepositoryReference)" + RemoveMetadata="MSBuildSourceProjectFile;MSBuildSourceTargetName;OriginalItemSpec" /> + From 542099329f345589ab149c1e20eb61d862003b22 Mon Sep 17 00:00:00 2001 From: Matt Thalman Date: Thu, 22 Feb 2024 09:11:59 -0600 Subject: [PATCH 39/40] Remove unnecessary normalize calls --- .../content/repo-projects/package-source-build.proj | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 8baae7fc8b31..6949e69e1af8 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -45,14 +45,14 @@ - Date: Thu, 22 Feb 2024 09:17:01 -0600 Subject: [PATCH 40/40] Update src/SourceBuild/content/repo-projects/package-source-build.proj Co-authored-by: Viktor Hofer --- .../content/repo-projects/package-source-build.proj | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/SourceBuild/content/repo-projects/package-source-build.proj b/src/SourceBuild/content/repo-projects/package-source-build.proj index 6949e69e1af8..dc2858da4d2c 100644 --- a/src/SourceBuild/content/repo-projects/package-source-build.proj +++ b/src/SourceBuild/content/repo-projects/package-source-build.proj @@ -49,10 +49,10 @@ DestinationFolder="$(_SourceBuiltLayoutDir)" UseSymbolicLinksIfPossible="true" />