From fc785088a28006c774dcfe0916b69ff54e6f99c1 Mon Sep 17 00:00:00 2001
From: Viktor Hofer <viktor.hofer@microsoft.com>
Date: Tue, 10 Dec 2024 14:16:19 +0100
Subject: [PATCH] Set NetCurrent so that it doesn't roll forward automatically
 (#10622)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Set NetCurrent so that it doesn't roll forward automatically

The VSTest repo can't use a floating NetCurrent value as the NetCurrent TFM is actually hardcoded in a number of places => .nuspec files. Set it to net9.0 explicitly so that the build continues to work with a .NET 10 SDK and Arcade 10 SDK (which is used inside the VMR).

Don't hardcode the value when building from source as in that configuration, the TFM needs to actually target the very latest.

* Update Directory.Build.props

---------

Co-authored-by: Jakub Jareš <me@jakubjares.com>
---
 Directory.Build.props | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Directory.Build.props b/Directory.Build.props
index 94e7a2ac6d..778e8f488c 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -52,6 +52,8 @@
   <PropertyGroup>
     <!-- Naming is based on dotnet/runtime one -->
     <NetFrameworkMinimum>net462</NetFrameworkMinimum>
+    <!-- When not building from source, hardcode the NetCurrent TFM value as it is hardcoded in a number of places, i.e. nuspec files. -->
+    <NetCurrent Condition="'$(DotNetBuildSourceOnly)' != 'true'">net9.0</NetCurrent>
     <NetCoreAppMinimum>netcoreapp3.1</NetCoreAppMinimum>
     <!--
       Arcade is making things hard to work with. This property is unset when its value is equal to NetMinimum.