Skip to content

Increment NetCurrent and NetPrevious#14108

Closed
mthalman wants to merge 6 commits intodotnet:mainfrom
mthalman:netcurrent
Closed

Increment NetCurrent and NetPrevious#14108
mthalman wants to merge 6 commits intodotnet:mainfrom
mthalman:netcurrent

Conversation

@mthalman
Copy link
Member

No description provided.

@mthalman mthalman marked this pull request as draft October 11, 2023 19:21
@mthalman
Copy link
Member Author

I'm unsure of the rollout of this. It would force repos to take a dependency on a newer SDK in order to get the arcade dependency.

@mmitche
Copy link
Member

mmitche commented Oct 11, 2023

I'm unsure of the rollout of this. It would force repos to take a dependency on a newer SDK in order to get the arcade dependency.

I think we should wait on this a bit until the net9 SDK is stable. Then I would do the following:

  1. Update the SDK in arcade to net9
  2. Flow to repos
  3. Update Properties
  4. Flow to repos

Separating the new SDK usage from the tfm update should reduce some noise.

@ViktorHofer
Copy link
Member

Agreed. I think it's weeks too early to increment these TFM properties across the stack. Probably a month or so to let the stack first stabilize (i.e. dotnet/sdk and installer).

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change L40 to:

<NetToolMinimum>net8.0</NetToolMinimum>
<NetToolMinimum Condition="'$(NetToolMinimum)' == '$(NetToolCurrent)'" />

}
$dotnetPath = Join-Path $dotnetRoot (GetExecutableFileName 'dotnet')
$buildTool = @{ Path = $dotnetPath; Command = 'msbuild'; Tool = 'dotnet'; Framework = 'net8.0' }
$buildTool = @{ Path = $dotnetPath; Command = 'msbuild'; Tool = 'dotnet'; Framework = 'net9.0' }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this change needs to wait until the updated Arcade.Sdk is consumed in Arcade.

@ViktorHofer
Copy link
Member

I pushed to your branch to reset NetPrevious. Hope that's OK.

@mthalman
Copy link
Member Author

This is replaced by #14279.

@mthalman mthalman closed this Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants