Skip to content

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jan 6, 2023

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

  • Subscription: fd62beb8-99ae-4af6-3912-08d8e9750bf8
  • Build: 20230318.1
  • Date Produced: March 18, 2023 4:56:46 PM UTC
  • Commit: b12f035e893c34ec2c965d75f6e21b7a2667e98d
  • Branch: refs/heads/main

@oleksandr-didyk
Copy link
Contributor

@tmat pinging you here as a notice that the changes that are being flown-in with this Maestro PR are also affecting the #933 PR. Specifically:

  • the new Arcade version has new functionality that would allow better UX for source-building the repo in CI and locally
  • the PR for pre-built detection bumps the System.Text.Json version to latest, which in turn requires an undesirable warning suppression for targeting < net8.0

I would guess the best course of action would be to update sourcelink to target latest tfm. Could you please let me know if, in your opinion, that's the right choice in this context and if this work will be done by your or by someone else. Thanks!

@tmat
Copy link
Member

tmat commented Feb 21, 2023

@oleksandr-didyk

I tried to update net7.0 TFMs to $(NetCurrent), which should be net8.0 now.

I get the following error though:

D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102: Unable to find package Microsoft.WindowsDesktop.App.Runtime.win-x64 with version (= 8.0.0-alpha.1.23057.1) [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 110 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-eng [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-tools [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102: Unable to find package Microsoft.WindowsDesktop.App.Runtime.win-x86 with version (= 8.0.0-alpha.1.23057.1) [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 110 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-eng [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-tools [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.App.Runtime.win-x64 with version (= 8.0.0-alpha.1.23058.7) [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 113 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 1 version(s) in dotnet-tools [ Nearest version: 5.0.0-rc.2.20513.11 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-eng [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.App.Runtime.win-x86 with version (= 8.0.0-alpha.1.23058.7) [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 113 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 1 version(s) in dotnet-tools [ Nearest version: 5.0.0-rc.2.20513.11 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-eng [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.App.Runtime.osx-x64 with version (= 8.0.0-alpha.1.23058.7) [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 112 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 1 version(s) in dotnet-tools [ Nearest version: 5.0.0-rc.2.20513.11 ] [D:\sourcelink\SourceLink.sln]
D:\sourcelink\src\dotnet-sourcelink\dotnet-sourcelink.csproj : error NU1102:   - Found 0 version(s) in dotnet-eng [D:\sourcelink\SourceLink.sln]

@oleksandr-didyk
Copy link
Contributor

Those packages are coming from the dotnet8 feed, so you would need to add it to the NuGet.config

You can always find these assets yourself through the Darc CL tool -> https://github.com/dotnet/arcade/blob/main/Documentation/Darc.md

For example, here is how I found which NuGet feed is missing:

darc get-asset --name "Microsoft.AspNetCore.App.Runtime.win-x64" --version "8.0.0-alpha.1.23058.7" --max-age 60

with the output:

Looking up assets with name 'Microsoft.AspNetCore.App.Runtime.win-x64' and version '8.0.0-alpha.1.23058.7' in the last 60 days
Microsoft.AspNetCore.App.Runtime.win-x64 @ 8.0.0-alpha.1.23058.7
Repository:    https://github.com/dotnet/aspnetcore
Branch:        main
Commit:        cefc6ccf4b23ac9f07418cd10e374cf533276eee
Build Number:  20230108.7
Date Produced: 1/9/2023 1:58 AM
Build Link:    https://dev.azure.com/dnceng/internal/_build/results?buildId=2082781
AzDO Build Id: 2082781
BAR Build Id:  162010
Released:      False
Channels:
- .NET 8
Locations:
- https://dev.azure.com/dnceng/internal/_apis/build/builds/2082781/artifacts (Container)
- https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet8/nuget/v3/index.json (NugetFeed)

@tmat
Copy link
Member

tmat commented Feb 23, 2023

Shouldn't Arcade automatically update nuget.config and add the right feed when bumping to new .NET version? Seems like the goals is to automate bumping versions as much as possible. While setting $(CurrentNet) to net8.0 addresses that for TFM it will still need manual intervention unless nuget.config is also updated automatically.

@tmat tmat force-pushed the darc-main-de856c79-f46a-447a-8771-6f43fe5187d1 branch from e2540ad to 3f185ab Compare February 24, 2023 19:25
@mmitche
Copy link
Member

mmitche commented Mar 7, 2023

Shouldn't Arcade automatically update nuget.config and add the right feed when bumping to new .NET version? Seems like the goals is to automate bumping versions as much as possible. While setting $(CurrentNet) to net8.0 addresses that for TFM it will still need manual intervention unless nuget.config is also updated automatically.

Arcade does not update any set of feeds except for stable package dependency flow.

@mmitche
Copy link
Member

mmitche commented Mar 7, 2023

@tmat Is the netcoreapp3.1 target still intended in this repo, since it's EOL now?

@oleksandr-didyk
Copy link
Contributor

oleksandr-didyk commented Mar 14, 2023

@tmat - I apologize for the ping, but would you have any update on the progress for this PR? Or maybe there is something that we can help with to push it further? Thank you!

@tmat
Copy link
Member

tmat commented Apr 5, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

dotnet-maestro bot and others added 10 commits April 4, 2023 21:16
…105.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23055.3
…108.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23058.1
…113.7

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23063.7
…117.5

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23067.5
…127.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23077.1
…203.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23103.1
…210.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23110.3
…218.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22630.1 -> To Version 8.0.0-beta.23118.1
@tmat tmat force-pushed the darc-main-de856c79-f46a-447a-8771-6f43fe5187d1 branch from 863a7a6 to e7788ce Compare April 5, 2023 04:16
@tmat
Copy link
Member

tmat commented Apr 5, 2023

@oleksandr-didyk @mmitche Seems to be mostly working now. Some source build issue.

@oleksandr-didyk
Copy link
Contributor

oleksandr-didyk commented Apr 5, 2023

The failure happens both in normal and source-build and seems to be related to this change -> e7788ce (#936)

The TFM default var is probably wrong - instead of $(CurrentNet) it should be $(NetCurrent) (https://github.com/dotnet/arcade/blob/8bbaaa29ff764792da8518d2ad708ee4dd3cf3d5/src/Microsoft.DotNet.Arcade.Sdk/tools/TargetFrameworkDefaults.props#L17) just like in other projects

@tmat
Copy link
Member

tmat commented Apr 5, 2023

@oleksandr-didyk Oh, thanks! Fixing.

@dotnet-maestro dotnet-maestro bot merged commit 8514735 into main Apr 5, 2023
@dotnet-maestro dotnet-maestro bot deleted the darc-main-de856c79-f46a-447a-8771-6f43fe5187d1 branch April 5, 2023 17:11
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