|
| 1 | +From d5f8588d6a97ba9972e572219c2a1a714fd587b1 Mon Sep 17 00:00:00 2001 |
| 2 | +From: Davis Goodin <dagood@microsoft.com> |
| 3 | +Date: Wed, 13 Jun 2018 12:35:52 -0500 |
| 4 | +Subject: [PATCH] Add versioned AspNetCore patch override properties |
| 5 | + |
| 6 | +https://github.com/aspnet/websdk/issues/355 will include changes that make this patch unnecessary. |
| 7 | +--- |
| 8 | + .../Sdk.DefaultItems.targets | 10 ++++++---- |
| 9 | + 1 file changed, 6 insertions(+), 4 deletions(-) |
| 10 | + |
| 11 | +diff --git a/src/Web/Microsoft.NET.Sdk.Web.Targets/Sdk.DefaultItems.targets b/src/Web/Microsoft.NET.Sdk.Web.Targets/Sdk.DefaultItems.targets |
| 12 | +index 9fc331e..15ebdc5 100644 |
| 13 | +--- a/src/Web/Microsoft.NET.Sdk.Web.Targets/Sdk.DefaultItems.targets |
| 14 | ++++ b/src/Web/Microsoft.NET.Sdk.Web.Targets/Sdk.DefaultItems.targets |
| 15 | +@@ -67,11 +67,12 @@ Copyright (c) .NET Foundation. All rights reserved. |
| 16 | + <!-- Latest patch version of .All Framework --> |
| 17 | + <PropertyGroup Condition="'$(LatestAspNetCoreAllPatchVersion)' == ''"> |
| 18 | + <!-- Placeholder for setting latest patch version using the bundled version from CLI for previous TFMs, this will not apply until 2.2 --> |
| 19 | +- <!-- <LatestAspNetCoreAllPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.1'">$(LatestPatchVersionForAspNetCoreAll2_1)</LatestAspNetCoreAllPatchVersion> --> |
| 20 | ++ <LatestAspNetCoreAllPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.1'">$(LatestPatchVersionForAspNetCoreAll2_1)</LatestAspNetCoreAllPatchVersion> |
| 21 | ++ <LatestAspNetCoreAllPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.2'">$(LatestPatchVersionForAspNetCoreAll2_2)</LatestAspNetCoreAllPatchVersion> |
| 22 | + |
| 23 | + <!-- If targeting the same release that is bundled with the .NET Core SDK, use the bundled package version |
| 24 | + provided by Microsoft.NETCoreSdk.BundledVersions.props --> |
| 25 | +- <LatestAspNetCoreAllPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '$(BundledAspNetCoreAllTargetFrameworkVersion)'">$(BundledAspNetCoreAllPackageVersion)</LatestAspNetCoreAllPatchVersion> |
| 26 | ++ <LatestAspNetCoreAllPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '$(BundledAspNetCoreAllTargetFrameworkVersion)' AND '$(LatestAspNetCoreAllPatchVersion)' == ''">$(BundledAspNetCoreAllPackageVersion)</LatestAspNetCoreAllPatchVersion> |
| 27 | + <!-- If not covered by the previous cases use the target framework version for the latest patch version --> |
| 28 | + <LatestAspNetCoreAllPatchVersion Condition="'$(LatestAspNetCoreAllPatchVersion)' == ''">$(_TargetFrameworkVersionWithoutV)</LatestAspNetCoreAllPatchVersion> |
| 29 | + </PropertyGroup> |
| 30 | +@@ -79,11 +80,12 @@ Copyright (c) .NET Foundation. All rights reserved. |
| 31 | + <!-- Latest patch version of .App Framework --> |
| 32 | + <PropertyGroup Condition="'$(LatestAspNetCoreAppPatchVersion)' == ''"> |
| 33 | + <!-- Placeholder for setting latest patch version using the bundled version from CLI for previous TFMs, this will not apply until 2.2 --> |
| 34 | +- <!-- <LatestAspNetCoreAppPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.1'">$(LatestPatchVersionForAspNetCoreApp2_1)</LatestAspNetCoreAppPatchVersion> --> |
| 35 | ++ <LatestAspNetCoreAppPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.1'">$(LatestPatchVersionForAspNetCoreApp2_1)</LatestAspNetCoreAppPatchVersion> |
| 36 | ++ <LatestAspNetCoreAppPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '2.2'">$(LatestPatchVersionForAspNetCoreApp2_2)</LatestAspNetCoreAppPatchVersion> |
| 37 | + |
| 38 | + <!-- If targeting the same release that is bundled with the .NET Core SDK, use the bundled package version |
| 39 | + provided by Microsoft.NETCoreSdk.BundledVersions.props --> |
| 40 | +- <LatestAspNetCoreAppPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '$(BundledAspNetCoreAppTargetFrameworkVersion)'">$(BundledAspNetCoreAppPackageVersion)</LatestAspNetCoreAppPatchVersion> |
| 41 | ++ <LatestAspNetCoreAppPatchVersion Condition="'$(_TargetFrameworkVersionWithoutV)' == '$(BundledAspNetCoreAppTargetFrameworkVersion)' AND '$(LatestAspNetCoreAppPatchVersion)' == ''">$(BundledAspNetCoreAppPackageVersion)</LatestAspNetCoreAppPatchVersion> |
| 42 | + <!-- If not covered by the previous cases use the target framework version for the latest patch version --> |
| 43 | + <LatestAspNetCoreAppPatchVersion Condition="'$(LatestAspNetCoreAppPatchVersion)' == ''">$(_TargetFrameworkVersionWithoutV)</LatestAspNetCoreAppPatchVersion> |
| 44 | + </PropertyGroup> |
| 45 | +-- |
| 46 | +2.17.1.windows.2 |
| 47 | + |
0 commit comments