Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[main] Update dependencies from dotnet/arcade #8762

Merged
merged 35 commits into from
Jun 23, 2023

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented May 25, 2023

This pull request updates the following dependencies

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

  • Subscription: a0cb50dc-7d38-4a32-e166-08db18b91f66
  • Build: 20230620.3
  • Date Produced: June 20, 2023 9:04:29 PM UTC
  • Commit: 06d73ccc1f421af06c2794741a9913f474f66b3d
  • Branch: refs/heads/main

Coherency Updates

The following updates ensure that dependencies with a CoherentParentDependency
attribute were produced in a build used as input to the parent dependency's build.
See Dependency Description Format

  • Coherency Updates:
    • Microsoft.SourceLink.GitHub: from 8.0.0-beta.23252.2 to 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk)
    • Microsoft.DotNet.XliffTasks: from 1.0.0-beta.23266.1 to 1.0.0-beta.23312.2 (parent: Microsoft.DotNet.Arcade.Sdk)

…524.6

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23274.6
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approving dependency update.

@333fred 333fred mentioned this pull request May 25, 2023
dotnet-maestro bot added 7 commits May 26, 2023 13:07
…525.2

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23275.2

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23272.1 (parent: Microsoft.DotNet.Arcade.Sdk
…526.2

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23276.2

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23272.1 (parent: Microsoft.DotNet.Arcade.Sdk
…527.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23277.1

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23272.1 (parent: Microsoft.DotNet.Arcade.Sdk
…529.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23279.1

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23272.1 (parent: Microsoft.DotNet.Arcade.Sdk
…529.4

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23279.4

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23272.1 (parent: Microsoft.DotNet.Arcade.Sdk
…530.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23280.3

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23279.1 (parent: Microsoft.DotNet.Arcade.Sdk
…531.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23281.3

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23279.1 (parent: Microsoft.DotNet.Arcade.Sdk
@davidwengier
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

333fred and others added 10 commits June 2, 2023 12:05
…602.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23302.3

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23279.1 (parent: Microsoft.DotNet.Arcade.Sdk
…607.2

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23307.2

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23305.1 (parent: Microsoft.DotNet.Arcade.Sdk
…608.2

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23308.2

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23305.1 (parent: Microsoft.DotNet.Arcade.Sdk
…609.8

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23309.8

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23266.1 -> To Version 1.0.0-beta.23305.1 (parent: Microsoft.DotNet.Arcade.Sdk
…613.6

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23313.6

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23309.3 (parent: Microsoft.DotNet.Arcade.Sdk
…614.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23314.1

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23309.3 (parent: Microsoft.DotNet.Arcade.Sdk
@chsienki
Copy link
Contributor

Skipping failing tests, tracked by #7982

@333fred
Copy link
Member

333fred commented Jun 15, 2023

@dotnet/razor-tooling this is now failing with a build error in the tooling layer, could you please take a look?

@jaredpar
Copy link
Member

@baronfel

D:\a\_work\1\s\.dotnet\sdk\8.0.100-preview.5.23303.2\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(254,5): error NETSDK1201: For projects targeting .NET 8.0 and higher, specifying a RuntimeIdentifier will no longer produce a self contained app by default. To continue building self-contained apps, set the SelfContained property to true or use the --self-contained argument. [D:\a\_work\1\s\src\Razor\src\rzls\rzls.csproj::TargetFramework=net7.0]
##[error].dotnet\sdk\8.0.100-preview.5.23303.2\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(254,5): error NETSDK1201: (NETCORE_ENGINEERING_TELEMETRY=Build) For projects targeting .NET 8.0 and higher, specifying a RuntimeIdentifier will no longer produce a self contained app by default. To continue building self-contained apps, set the SelfContained property to true or use the --self-contained argument.
D:\a\_work\1\s\.dotnet\sdk\8.0.100-preview.5.23303.2\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(254,5): error NETSDK1201: For projects targeting .NET 8.0 and higher, specifying a RuntimeIdentifier will no longer produce a self contained app by default. To continue building self-contained apps, set the SelfContained property to true or use the --self-contained argument. [D:\a\_work\1\s\src\Razor\src\rzls\rzls.csproj::TargetFramework=net7.0]
##[error].dotnet\sdk\8.0.100-preview.5.23303.2\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(254,5): error NETSDK1201: (NETCORE_ENGINEERING_TELEMETRY=Build) For projects targeting .NET 8.0 and higher, specifying a RuntimeIdentifier will no longer produce a self contained app by default. To continue building self-contained apps, set the SelfContained property to true or use the --self-contained argument.
  Microsoft.AspNetCore.Razor.Utilities.Shared -> D:\a\_work\1\s\artifacts\bin\Microsoft.AspNetCore.Razor.Utilities.Shared\Release\net7.0\Microsoft.AspNetCore.Razor.Utilities.Shared.dll

Thought this was changed to not have a diagnostic

@maryamariyan
Copy link
Member

If we can get unblocked with this PR and get latest arcade bits that would fix specific component governance issues that we are tracking to be done by end of June

…616.6

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23316.6

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk
@@ -6,6 +6,7 @@
<OutputType>Exe</OutputType>
<Description>Razor is a markup syntax for adding server-side logic to web pages. This package contains a Razor language server.</Description>
<EnableApiCheck>false</EnableApiCheck>
<TreatWarningsAsErrors>false</TreatWarningsAsErrors>
Copy link
Member

@jjonescz jjonescz Jun 19, 2023

Choose a reason for hiding this comment

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

@jaredpar to suppress the warning, you can do this:

Suggested change
<TreatWarningsAsErrors>false</TreatWarningsAsErrors>
<MSBuildWarningsAsMessages>NETSDK1201</MSBuildWarningsAsMessages>

…619.1

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23319.1

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk
@nagilson
Copy link
Member

We were able to resolve this friday evening by setting -p:_SelfContainedWasSpecified=true

dotnet-maestro bot and others added 8 commits June 20, 2023 13:06
…619.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23319.3

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk
…620.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23320.3

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk
…620.3

Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.23269.2 -> To Version 8.0.0-beta.23320.3

Dependency coherency updates

Microsoft.SourceLink.GitHub,Microsoft.DotNet.XliffTasks
 From Version 8.0.0-beta.23252.2 -> To Version 8.0.0-beta.23314.2 (parent: Microsoft.DotNet.Arcade.Sdk
jaredpar and others added 3 commits June 22, 2023 13:51
Co-authored-by: Chet Husk <baronfel@users.noreply.github.com>
Copy link
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

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

these changes are what I needed to build on Ubuntu

src/Razor/src/rzls/rzls.csproj Outdated Show resolved Hide resolved
src/Razor/src/rzls/rzls.csproj Outdated Show resolved Hide resolved
Co-authored-by: Chet Husk <baronfel@users.noreply.github.com>
@@ -1,7 +1,7 @@
@echo off
setlocal

set _args=-restore -build /p:TreatWarningsAsErrors=true %*
set _args=-restore -build /p:_SelfContainedWasSpecified=true %*
Copy link
Member

Choose a reason for hiding this comment

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

Is this change still needed? There don't seem to be any warnings to suppress in the latest CI builds.

Copy link
Member

Choose a reason for hiding this comment

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

I will revert this and the other change you mentioned after merging. This finally went greenand I want to get this in to unblock other items. Those changes dont' really impact the semantics so should be fine to be in for a few hour.s

@@ -6,6 +6,7 @@
<OutputType>Exe</OutputType>
<Description>Razor is a markup syntax for adding server-side logic to web pages. This package contains a Razor language server.</Description>
<EnableApiCheck>false</EnableApiCheck>
<TreatWarningsAsErrors>false</TreatWarningsAsErrors>
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@jaredpar jaredpar merged commit 203d11f into main Jun 23, 2023
@jaredpar jaredpar deleted the darc-main-0b91fd41-3be2-4e1e-81e0-c48d3d238f51 branch June 23, 2023 18:11
jaredpar added a commit that referenced this pull request Jun 23, 2023
This addresses some feedback from #8762
jaredpar added a commit that referenced this pull request Jun 23, 2023
This addresses some feedback from #8762
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants