Revert "GH-2095 - Target .net48 instead of .net472"

arturcic · arturcic · commit b22e8726683a · 2020-06-28T09:30:44.000+03:00
This reverts commit 5330bbb.
diff --git a/build/pack.cake b/build/pack.cake
@@ -4,7 +4,7 @@ Task("Pack-Prepare")
 {
     PackPrepareNative(Context, parameters);
 
-    var frameworks = new[] { parameters.CoreFxVersion21, parameters.FullFxVersion48 };
+    var frameworks = new[] { parameters.CoreFxVersion21, parameters.FullFxVersion472 };
 
     // MsBuild Task
     foreach(var framework in frameworks)
diff --git a/build/test.cake b/build/test.cake
@@ -5,7 +5,7 @@ Task("UnitTest")
     .IsDependentOn("Build")
     .Does<BuildParameters>((parameters) =>
 {
-    var frameworks = new[] { parameters.CoreFxVersion31, parameters.FullFxVersion48 };
+    var frameworks = new[] { parameters.CoreFxVersion31, parameters.FullFxVersion472 };
     var testResultsPath = parameters.Paths.Directories.TestResultsOutput;
 
     foreach(var framework in frameworks)
@@ -39,7 +39,7 @@ Task("UnitTest")
                     Exclude = new List<string> { "[GitVersion*.Tests]*", "[GitVersionTask.MsBuild]*" }
                 };
 
-                if (IsRunningOnUnix() && string.Equals(framework, parameters.FullFxVersion48))
+                if (IsRunningOnUnix() && string.Equals(framework, parameters.FullFxVersion472))
                 {
                     settings.Filter = "TestCategory!=NoMono";
                 }
diff --git a/build/utils/parameters.cake b/build/utils/parameters.cake
@@ -12,7 +12,7 @@ public class BuildParameters
     public const string MainRepoName = "GitVersion";
     public string CoreFxVersion21 { get; private set; }  = "netcoreapp2.1";
     public string CoreFxVersion31 { get; private set; }  = "netcoreapp3.1";
-    public string FullFxVersion48 { get; private set; }  = "net48";
+    public string FullFxVersion472 { get; private set; } = "net472";
 
     public string DockerDistro { get; private set; }
     public string DockerDotnetVersion { get; private set; }
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -66,7 +66,7 @@
         </None>
     </ItemGroup>
 
-    <ItemGroup Condition=" '$(TargetFramework)' == 'net48' ">
+    <ItemGroup Condition=" '$(TargetFramework)' == 'net472' ">
         <PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0" PrivateAssets="All" />
     </ItemGroup>
 </Project>
diff --git a/src/Docker/Readme.md b/src/Docker/Readme.md
@@ -1,17 +1,17 @@
 # There are 4 variants of docker image
 
-- based on **microsoft/dotnet-framework:4.8-runtime** - Windows Full FX
+- based on **microsoft/dotnet-framework:4.7.2-runtime** - Windows Full FX
 - based on **microsoft/dotnet:2.1-runtime** - Windows dotnet core
 - based on **mono:5.18** - Linux Full FX - on mono
 - based on **microsoft/dotnet:2.1-runtime** - Linux dotnet core
 - based on **centos:7** - linux dotnet core
 
 To run on windows container run this
-`docker run --rm -v "$(pwd):c:/repo" gittools/gitversion:latest-windows-net48 c:/repo`
+`docker run --rm -v "$(pwd):c:/repo" gittools/gitversion:latest-windows-net472 c:/repo`
 
 `docker run --rm -v "$(pwd):c:/repo" gittools/gitversion:latest-windows-netcoreapp2.1 c:/repo`
 
 To run on linux container run this
-`docker run --rm -v "$(pwd):/repo" gittools/gitversion:latest-linux-net48 /repo`
+`docker run --rm -v "$(pwd):/repo" gittools/gitversion:latest-linux-net472 /repo`
 
 `docker run --rm -v "$(pwd):/repo" gittools/gitversion:latest-linux-netcoreapp2.1 /repo`
diff --git a/src/GitVersionCore.Tests/GitVersionCore.Tests.csproj b/src/GitVersionCore.Tests/GitVersionCore.Tests.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
     <PropertyGroup>
-        <TargetFrameworks>net48;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
+        <TargetFrameworks>net472;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
 
         <DebugType>full</DebugType>
         <Optimize>false</Optimize>
diff --git a/src/GitVersionCore/GitVersionCore.csproj b/src/GitVersionCore/GitVersionCore.csproj
@@ -14,15 +14,17 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="System.Net.Requests" Version="4.3.0" />
-
         <PackageReference Include="LibGit2Sharp" Version="$(PackageVersion_LibGit2Sharp)" />
         <PackageReference Include="JetBrains.Annotations" Version="$(PackageVersion_JetBrainsAnnotations)" PrivateAssets="All" />
         <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="$(PackageVersion_MicrosoftExtensions)" />
         <PackageReference Include="Microsoft.Extensions.Options" Version="$(PackageVersion_MicrosoftExtensions)" />
         <PackageReference Include="YamlDotNet" Version="$(PackageVersion_YamlDotNet)" />
     </ItemGroup>
 
+    <ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
+        <PackageReference Include="System.Net.Requests" Version="4.3.0" />
+    </ItemGroup>
+
     <ItemGroup>
         <Compile Remove="VersionConverters\*\AddFormats\**\*.*" />
         <Compile Remove="VersionConverters\*\Templates\**\*.*" />
diff --git a/src/GitVersionExe.Tests/GitVersionExe.Tests.csproj b/src/GitVersionExe.Tests/GitVersionExe.Tests.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
     <PropertyGroup>
-        <TargetFrameworks>net48;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
+        <TargetFrameworks>net472;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
 
         <IsTestProject>true</IsTestProject>
     </PropertyGroup>
diff --git a/src/GitVersionExe/GitVersionExe.csproj b/src/GitVersionExe/GitVersionExe.csproj
@@ -12,7 +12,7 @@
     </PropertyGroup>
 
     <PropertyGroup Condition=" '$(PackAsTool)' != 'true'">
-        <TargetFrameworks>net48;$(TargetFrameworks);</TargetFrameworks>
+        <TargetFrameworks>net472;$(TargetFrameworks);</TargetFrameworks>
     </PropertyGroup>
 
     <PropertyGroup Condition=" '$(PackAsTool)' == 'true' ">
@@ -21,7 +21,7 @@
         <PackageDescription>Derives SemVer information from a repository following GitFlow or GitHubFlow. This is the .NET Core Global Tool allowing usage of GitVersion from command line.</PackageDescription>
     </PropertyGroup>
 
-    <ItemGroup Condition="'$(TargetFramework)'=='net48'">
+    <ItemGroup Condition="'$(TargetFramework)'=='net472'">
         <PackageReference Include="JetBrains.Annotations" Version="$(PackageVersion_JetBrainsAnnotations)" />
         <PackageReference Include="LibGit2Sharp.NativeBinaries" Version="$(PackageVersion_LibGit2Sharp_NativeBinaries)" />
     </ItemGroup>
diff --git a/src/GitVersionTask.MsBuild/GitVersionTask.MsBuild.csproj b/src/GitVersionTask.MsBuild/GitVersionTask.MsBuild.csproj
@@ -1,7 +1,7 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 
     <PropertyGroup>
-        <TargetFrameworks>net48;netstandard2.0</TargetFrameworks>
+        <TargetFrameworks>net472;netstandard2.0</TargetFrameworks>
         <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
     </PropertyGroup>
 
diff --git a/src/GitVersionTask.MsBuild/LibGit2Sharp/LibGit2SharpLoader.cs b/src/GitVersionTask.MsBuild/LibGit2Sharp/LibGit2SharpLoader.cs
@@ -4,7 +4,7 @@
 using System.IO;
 using System.Reflection;
 
-#if NETFRAMEWORK
+#if NET472
 using System;
 using System.Collections.Generic;
 #endif
diff --git a/src/GitVersionTask.MsBuild/PlatformAbstraction/Native/NativeMethods.Unix.cs b/src/GitVersionTask.MsBuild/PlatformAbstraction/Native/NativeMethods.Unix.cs
@@ -2,7 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
-#if NETFRAMEWORK
+#if NET45
 
 using System;
 using System.Runtime.InteropServices;
diff --git a/src/GitVersionTask.Tests/GitVersionTask.Tests.csproj b/src/GitVersionTask.Tests/GitVersionTask.Tests.csproj
@@ -1,6 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
     <PropertyGroup>
-        <TargetFrameworks>net48;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
+        <TargetFrameworks>net472;netcoreapp2.1;netcoreapp3.1</TargetFrameworks>
 
         <IsTestProject>true</IsTestProject>
     </PropertyGroup>
diff --git a/src/GitVersionTask.Tests/Helpers/MsBuildExeFixture.cs b/src/GitVersionTask.Tests/Helpers/MsBuildExeFixture.cs
@@ -67,7 +67,7 @@ public void CreateTestProject(Action<ProjectCreator> extendProject)
             var project = RuntimeFramework.CurrentFramework.Runtime switch
             {
                 RuntimeType.NetCore => ProjectCreator.Templates.SdkCsproj(ProjectPath),
-                RuntimeType.Net => ProjectCreator.Templates.LegacyCsproj(ProjectPath, defaultTargets: null, targetFrameworkVersion: "v4.8", toolsVersion: "15.0"),
+                RuntimeType.Net => ProjectCreator.Templates.LegacyCsproj(ProjectPath, defaultTargets: null, targetFrameworkVersion: "v4.7.2", toolsVersion: "15.0"),
                 _ => null
             };
 
diff --git a/src/GitVersionTask/GitVersionTask.csproj b/src/GitVersionTask/GitVersionTask.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
     <PropertyGroup>
-        <TargetFrameworks>net48;netcoreapp2.1</TargetFrameworks>
+        <TargetFrameworks>net472;netcoreapp2.1</TargetFrameworks>
 
         <PackageId>GitVersionTask</PackageId>
         <Title>GitVersionTask</Title>
diff --git a/src/GitVersionTask/build/GitVersionTask.props b/src/GitVersionTask/build/GitVersionTask.props
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
     <PropertyGroup>
-        <GitVersionAssemblyFile Condition="$(GitVersionAssemblyFile) == '' And '$(MSBuildRuntimeType)' != 'Core'">$(MSBuildThisFileDirectory)..\tools\net48\GitVersionTask.MsBuild.dll</GitVersionAssemblyFile>
+        <GitVersionAssemblyFile Condition="$(GitVersionAssemblyFile) == '' And '$(MSBuildRuntimeType)' != 'Core'">$(MSBuildThisFileDirectory)..\tools\net472\GitVersionTask.MsBuild.dll</GitVersionAssemblyFile>
         <GitVersionAssemblyFile Condition="$(GitVersionAssemblyFile) == '' And '$(MSBuildRuntimeType)' == 'Core'">$(MSBuildThisFileDirectory)..\tools\netstandard2.0\GitVersionTask.MsBuild.dll</GitVersionAssemblyFile>
 
         <SolutionDir Condition="$(SolutionDir) == '' Or $(SolutionDir) == '*Undefined*'">$(MSBuildProjectDirectory)\..\</SolutionDir>
diff --git a/src/GitVersionTask/nuget-files.props b/src/GitVersionTask/nuget-files.props
@@ -1,9 +1,9 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
     <PropertyGroup>
-        <FullPublishTask>bin/$(Configuration)/net48</FullPublishTask>
+        <FullPublishTask>bin/$(Configuration)/net472</FullPublishTask>
         <CorePublishTask>bin/$(Configuration)/netcoreapp2.1/publish</CorePublishTask>
-        <FullTarget>tools/net48</FullTarget>
+        <FullTarget>tools/net472</FullTarget>
         <CoreTarget>tools/netstandard2.0</CoreTarget>
     </PropertyGroup>
     <ItemGroup>
diff --git a/tests/integration/full/app.csproj b/tests/integration/full/app.csproj
@@ -5,7 +5,7 @@
     <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
     <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
     <ProjectGuid>{C1765652-F8AE-4FFD-AFC8-72054A5A91B7}</ProjectGuid>
-    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.7.2</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>
     <Deterministic>true</Deterministic>
@@ -14,15 +14,15 @@
     <PlatformTarget>AnyCPU</PlatformTarget>
     <DebugSymbols>true</DebugSymbols>
     <DebugType>full</DebugType>
-    <Optimize>false</Optimize>
+    <Optimize>false</Optimize>    
     <DefineConstants>DEBUG;TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
     <PlatformTarget>AnyCPU</PlatformTarget>
     <DebugType>pdbonly</DebugType>
-    <Optimize>true</Optimize>
+    <Optimize>true</Optimize>    
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ Task("Pack-Prepare")`
`4`	`4`	`{`
`5`	`5`	`PackPrepareNative(Context, parameters);`
`6`	`6`
`7`		`- var frameworks = new[] { parameters.CoreFxVersion21, parameters.FullFxVersion48 };`
	`7`	`+ var frameworks = new[] { parameters.CoreFxVersion21, parameters.FullFxVersion472 };`
`8`	`8`
`9`	`9`	`// MsBuild Task`
`10`	`10`	`foreach(var framework in frameworks)`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ Task("UnitTest")`
`5`	`5`	`.IsDependentOn("Build")`
`6`	`6`	`.Does<BuildParameters>((parameters) =>`
`7`	`7`	`{`
`8`		`- var frameworks = new[] { parameters.CoreFxVersion31, parameters.FullFxVersion48 };`
	`8`	`+ var frameworks = new[] { parameters.CoreFxVersion31, parameters.FullFxVersion472 };`
`9`	`9`	`var testResultsPath = parameters.Paths.Directories.TestResultsOutput;`
`10`	`10`
`11`	`11`	`foreach(var framework in frameworks)`
`@@ -39,7 +39,7 @@ Task("UnitTest")`
`39`	`39`	`Exclude = new List<string> { "[GitVersion.Tests]", "[GitVersionTask.MsBuild]*" }`
`40`	`40`	`};`
`41`	`41`
`42`		`- if (IsRunningOnUnix() && string.Equals(framework, parameters.FullFxVersion48))`
	`42`	`+ if (IsRunningOnUnix() && string.Equals(framework, parameters.FullFxVersion472))`
`43`	`43`	`{`
`44`	`44`	`settings.Filter = "TestCategory!=NoMono";`
`45`	`45`	`}`