Skip to content

Commit d77955f

Browse files
authored
Make SourceBuild.props work on Windows
Contributes to VMR work: dotnet/source-build#3926 Changes are from dotnet/dotnet#46
1 parent cc9cff3 commit d77955f

File tree

1 file changed

+16
-7
lines changed

1 file changed

+16
-7
lines changed

eng/SourceBuild.props

+16-7
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,11 @@
88
<SourceBuildTargetFrameworkFilter>netstandard2.0%3bnetstandard2.1%3bnetcoreapp2.1%3bnetcoreapp3.1%3bnet5.0%3bnet6.0%3bnet7.0%3bnet8.0%3bnet9.0</SourceBuildTargetFrameworkFilter>
99
</PropertyGroup>
1010

11+
<!-- Only run this target in source-build only mode. -->
1112
<Target Name="PrepareGlobalJsonForSourceBuild"
1213
AfterTargets="PrepareInnerSourceBuildRepoRoot"
13-
BeforeTargets="RunInnerSourceBuildCommand">
14+
BeforeTargets="RunInnerSourceBuildCommand"
15+
Condition="'$(ArcadeBuildFromSource)' == 'true'">
1416
<Exec
1517
Command="./eng/scripts/prepare-sourcebuild-globaljson.sh"
1618
WorkingDirectory="$(InnerSourceBuildRepoRoot)" />
@@ -28,13 +30,11 @@
2830
<Delete Files="$(InnerSourceBuildRepoRoot).globalconfig" />
2931
</Target>
3032

31-
<!--
32-
Build RepoTasks - this is normally triggered via the build script but the inner ArPow source-build is run via msbuild
33-
-->
33+
<!-- Build RepoTasks - this is normally triggered via the build script but the inner ArPow source-build is run via msbuild.
34+
https://github.com/dotnet/source-build/issues/3807 -->
3435
<Target Name="BuildRepoTasks"
3536
Condition="'$(ArcadeInnerBuildFromSource)' == 'true'"
3637
BeforeTargets="Execute">
37-
3838
<!-- If the alternative runtime location and key are present, pass those through -->
3939
<PropertyGroup>
4040
<_AdditionalRepoTaskBuildArgs />
@@ -45,7 +45,8 @@
4545
<ItemGroup>
4646
<!-- We need to flow FullAssemblySigningSupported even when building repo tasks because they use full signing -->
4747
<InnerBuildEnv Condition="'$(FullAssemblySigningSupported)' != ''" Include="FullAssemblySigningSupported=$(FullAssemblySigningSupported)" />
48-
<InnerBuildEnv Include="DotNetBuildFromSource=true" />
48+
<InnerBuildEnv Include="DotNetBuildFromSource=$(ArcadeBuildFromSource)" />
49+
<InnerBuildEnv Include="DotNetBuildVertical=$(ArcadeBuildVertical)" />
4950
</ItemGroup>
5051

5152
<!-- Call the build.sh script to build the repo tasks. Set IgnoreStandardErrorWarningFormat
@@ -56,7 +57,15 @@
5657
Command="./eng/build.sh --only-build-repo-tasks -bl $(_AdditionalRepoTaskBuildArgs)"
5758
IgnoreStandardErrorWarningFormat="true"
5859
WorkingDirectory="$(InnerSourceBuildRepoRoot)"
59-
EnvironmentVariables="@(InnerBuildEnv)" />
60+
EnvironmentVariables="@(InnerBuildEnv)"
61+
Condition="'$(OS)' != 'Windows_NT'" />
62+
63+
<Exec
64+
Command=".\eng\build.cmd -OnlyBuildRepoTasks -bl $(_AdditionalRepoTaskBuildArgs)"
65+
IgnoreStandardErrorWarningFormat="true"
66+
WorkingDirectory="$(InnerSourceBuildRepoRoot)"
67+
EnvironmentVariables="@(InnerBuildEnv)"
68+
Condition="'$(OS)' == 'Windows_NT'" />
6069
</Target>
6170

6271
<Target Name="CustomizeInnerBuildArgs"

0 commit comments

Comments
 (0)