Skip to content

Commit

Permalink
Merge pull request #1985 from dsplaisted/1071-rename-build-directory
Browse files Browse the repository at this point in the history
Rename SDK build directory to "targets" in source and layout
  • Loading branch information
dsplaisted authored Feb 23, 2018
2 parents acf05da + e301331 commit 6945e36
Show file tree
Hide file tree
Showing 37 changed files with 12 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@
<None Include="msbuildExtensions\**\*" PackagePath="msbuildExtensions\" />
<None Include="msbuildExtensions-ver\**\*" PackagePath="msbuildExtensions-ver\" />
<None Include="..\Common\Resources\xlf\**\*" LinkBase="Resources\xlf" />
<None Include="..\Common\build\**\*" PackagePath="msbuildExtensions\Microsoft\Microsoft.NET.Build.Extensions\" LinkBase="%(PackagePath)" />
<None Include="..\Common\targets\**\*" PackagePath="msbuildExtensions\Microsoft\Microsoft.NET.Build.Extensions\" LinkBase="%(PackagePath)" />
</ItemGroup>

<Target Name="PrepareAdditionalFilesToLayout" BeforeTargets="AssignTargetPaths">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,9 @@
</ItemGroup>

<ItemGroup>
<None Include="build\**\*" PackagePath="build\" />
<None Include="buildCrossTargeting\**\*" PackagePath="buildCrossTargeting\" />
<None Include="targets\**\*" PackagePath="targets\" />
<None Include="sdk\**\*" PackagePath="Sdk\" />
<None Include="..\Common\build\**\*" PackagePath="build\" LinkBase="build" />
<None Include="..\Common\targets\**\*" PackagePath="targets\" LinkBase="targets" />
<None Include="..\Common\Resources\xlf\**\*" LinkBase="Resources\xlf" />
</ItemGroup>

Expand All @@ -87,7 +86,7 @@
<Stage0SdkFile Include="$(_Stage0SdksFolder)\FSharp.NET.Sdk\**" SdkName="FSharp.NET.Sdk" />
<Stage0SdkFile Include="$(_Stage0SdksFolder)\Microsoft.NET.Sdk.Publish\**" SdkName="Microsoft.NET.Sdk.Publish" />
<Stage0SdkFile Include="$(_Stage0SdksFolder)\Microsoft.NET.Sdk.Web\**" SdkName="Microsoft.NET.Sdk.Web" />
<Stage0SdkFile Include="$(_Stage0SdksFolder)\Microsoft.NET.Sdk.Web.ProjectSystem\**" SdkName="Microsoft.NET.Sdk.Web.ProjectSystem"/>
<Stage0SdkFile Include="$(_Stage0SdksFolder)\Microsoft.NET.Sdk.Web.ProjectSystem\**" SdkName="Microsoft.NET.Sdk.Web.ProjectSystem" />
<LayoutFile Include="@(Stage0SdkFile)">
<TargetPath>..\%(Stage0SdkFile.SdkName)\%(Stage0SdkFile.RecursiveDir)%(Stage0SdkFile.Filename)%(Stage0SdkFile.Extension)</TargetPath>
</LayoutFile>
Expand Down
2 changes: 1 addition & 1 deletion src/Tasks/Microsoft.NET.Build.Tasks/sdk/Sdk.props
Original file line number Diff line number Diff line change
Expand Up @@ -27,5 +27,5 @@ Copyright (c) .NET Foundation. All rights reserved.
</PropertyGroup>

<Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" />
<Import Project="$(MSBuildThisFileDirectory)..\build\Microsoft.NET.Sdk.props" />
<Import Project="$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.props" />
</Project>
10 changes: 5 additions & 5 deletions src/Tasks/Microsoft.NET.Build.Tasks/sdk/Sdk.targets
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,15 @@ Copyright (c) .NET Foundation. All rights reserved.
<IsCrossTargetingBuild>true</IsCrossTargetingBuild>
</PropertyGroup>

<Import Project="$(MSBuildThisFileDirectory)..\build\Microsoft.NET.Sdk.BeforeCommonCrossTargeting.targets"
<Import Project="$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.BeforeCommonCrossTargeting.targets"
Condition="'$(IsCrossTargetingBuild)' == 'true'"/>
<Import Project="$(MSBuildThisFileDirectory)..\build\Microsoft.NET.Sdk.BeforeCommon.targets"
<Import Project="$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.BeforeCommon.targets"
Condition="'$(IsCrossTargetingBuild)' != 'true'"/>

<PropertyGroup Condition="'$(LanguageTargets)' == ''">
<LanguageTargets Condition="'$(MSBuildProjectExtension)' == '.csproj'">$(MSBuildToolsPath)\Microsoft.CSharp.targets</LanguageTargets>
<LanguageTargets Condition="'$(MSBuildProjectExtension)' == '.vbproj'">$(MSBuildToolsPath)\Microsoft.VisualBasic.targets</LanguageTargets>
<LanguageTargets Condition="'$(MSBuildProjectExtension)' == '.fsproj'">$(MSBuildThisFileDirectory)..\build\Microsoft.NET.Sdk.FSharp.targets</LanguageTargets>
<LanguageTargets Condition="'$(MSBuildProjectExtension)' == '.fsproj'">$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.FSharp.targets</LanguageTargets>

<!-- If LanguageTargets isn't otherwise set, then just import the common targets. This should allow the restore target to run,
which could bring in NuGet packages that set the LanguageTargets to something else. This means support for different
Expand All @@ -40,9 +40,9 @@ Copyright (c) .NET Foundation. All rights reserved.
using Sdk attribute (from .NET Core Sdk 1.0.0-preview4) -->
<Import Project="$(LanguageTargets)"/>

<Import Project="$(MSBuildThisFileDirectory)..\buildCrossTargeting\Microsoft.NET.Sdk.targets"
<Import Project="$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.CrossTargeting.targets"
Condition="'$(IsCrossTargetingBuild)' == 'true'"/>
<Import Project="$(MSBuildThisFileDirectory)..\build\Microsoft.NET.Sdk.targets"
<Import Project="$(MSBuildThisFileDirectory)..\targets\Microsoft.NET.Sdk.targets"
Condition="'$(IsCrossTargetingBuild)' != 'true'"/>

<!-- Import targets from NuGet.Build.Tasks.Pack package/Sdk -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ Copyright (c) .NET Foundation. All rights reserved.
<MSBuildAllProjects>$(MSBuildAllProjects);$(MSBuildThisFileFullPath)</MSBuildAllProjects>
</PropertyGroup>

<Import Project="..\build\Microsoft.NET.Sdk.Common.targets"/>
<Import Project="Microsoft.NET.Sdk.Common.targets"/>

<!--
============================================================
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ private CommandResult GenerateDepsAndRunTool(TestProject toolProject, [CallerMem

var args = new List<string>();

string generateDepsProjectPath = Path.Combine(TestContext.Current.ToolsetUnderTest.SdksPath, "Microsoft.NET.Sdk", "build", "GenerateDeps", "GenerateDeps.proj");
string generateDepsProjectPath = Path.Combine(TestContext.Current.ToolsetUnderTest.SdksPath, "Microsoft.NET.Sdk", "targets", "GenerateDeps", "GenerateDeps.proj");

args.Add($"/p:ProjectAssetsFile=\"{toolAssetsFilePath}\"");

Expand Down

0 comments on commit 6945e36

Please sign in to comment.