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

[release/6.4] Update dependencies from dotnet/core-setup #1223

Merged

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Sep 5, 2019

This pull request updates the following dependencies

From https://github.com/dotnet/core-setup

  • Build: 20190909.38
  • Date Produced: 9/10/2019 6:52 AM
  • Commit: 0d6763312a2754e45d604ebdc69c8ac7e21a2187
  • Branch: refs/heads/release/3.1
  • Updates:
    • Microsoft.NETCore.App.Ref -> 3.1.0-preview1.19459.38
    • Microsoft.NETCore.App.Runtime.win-x64 -> 3.1.0-preview1.19459.38

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

  • Microsoft.CSharp -> 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
  • System.CodeDom -> 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
  • System.ComponentModel.Annotations -> 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
  • System.Configuration.ConfigurationManager -> 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
  • System.Data.SqlClient -> 4.8.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)

@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Sep 5, 2019

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • Standard Merge Policies Unsuccessful checks: EntityFramework6-ci
  • ✔️ Standard Merge Policies Succeeded - No reviews have requested changes.

@dougbu dougbu added the area-infrastructure Issues related to our MSBuild projects and targets, build scripts, CI, etc. label Sep 7, 2019
…0190904.16

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19454.16
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19454.16

Dependency coherency updates

- Microsoft.CSharp - 4.7.0-rc1.19454.8 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.7.0-rc1.19454.8 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.7.0-rc1.19454.8 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.7.0-rc1.19454.8 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.8.0-rc1.19454.8 (parent: Microsoft.NETCore.App.Runtime.win-x64)
…0190905.31

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19455.31
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19455.31

Dependency coherency updates

- Microsoft.CSharp - 4.6.0-rc1.19455.6 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.6.0-rc1.19455.6 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.6.0-rc1.19455.6 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.6.0-rc1.19455.6 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.7.0-rc1.19455.6 (parent: Microsoft.NETCore.App.Runtime.win-x64)
…0190907.3

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19457.3
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19457.3

Dependency coherency updates

- Microsoft.CSharp - 4.7.0-preview1.19456.20 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.7.0-preview1.19456.20 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.7.0-preview1.19456.20 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.7.0-preview1.19456.20 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.8.0-preview1.19456.20 (parent: Microsoft.NETCore.App.Runtime.win-x64)
@dougbu dougbu force-pushed the darc-release/6.4-5b305db7-f7ad-4508-8982-a7a2c890614b branch from 555a6f2 to e1f6e29 Compare September 7, 2019 18:05
…0190907.10

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19457.10
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19457.10

Dependency coherency updates

- Microsoft.CSharp - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.8.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
@dougbu
Copy link
Member

dougbu commented Sep 7, 2019

Must retarget projects in this branch to 'netcoreapp3.1' to match the Microsoft.NETCore.App.Ref dependency.

@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Sep 7, 2019

Auto-Merge Status

This pull request has been merged because the following merge policies have succeeded.

  • ✔️ Standard Merge Policies Succeeded - Successful checks: EntityFramework6-ci
  • ✔️ Standard Merge Policies Succeeded - No reviews have requested changes.

@dougbu dougbu force-pushed the darc-release/6.4-5b305db7-f7ad-4508-8982-a7a2c890614b branch from 7e592aa to 6736dd5 Compare September 7, 2019 21:20
dotnet-maestro bot and others added 2 commits September 8, 2019 19:54
…0190908.7

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19458.7
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19458.7

Dependency coherency updates

- Microsoft.CSharp - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.7.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.8.0-preview1.19457.1 (parent: Microsoft.NETCore.App.Runtime.win-x64)
dotnet-maestro bot and others added 4 commits September 10, 2019 12:42
…0190909.38

- Microsoft.NETCore.App.Ref - 3.1.0-preview1.19459.38
- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview1.19459.38

Dependency coherency updates

- Microsoft.CSharp - 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.CodeDom - 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.ComponentModel.Annotations - 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Configuration.ConfigurationManager - 4.7.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
- System.Data.SqlClient - 4.8.0-preview1.19459.13 (parent: Microsoft.NETCore.App.Runtime.win-x64)
@dotnet-maestro dotnet-maestro bot merged commit 3220f15 into release/6.4 Sep 12, 2019
@dotnet-maestro dotnet-maestro bot deleted the darc-release/6.4-5b305db7-f7ad-4508-8982-a7a2c890614b branch September 12, 2019 00:03
@JunTaoLuo
Copy link
Contributor

@dougbu feel free to take a look at all of the changes I made but it looks like the bot merged this since all the checks passed.

<TargetingPackVersion Condition="'%(TargetFramework)' == 'netcoreapp3.0'">$(MicrosoftNETCoreAppRefPackageVersion)</TargetingPackVersion>
<!-- Use the latest version instead of the one bundled with the SDK. -->
<KnownFrameworkReference Update="Microsoft.NETCore.App"
Condition=" '$(TargetFramework)' == '$(DefaultNetCoreTargetFramework)' ">
Copy link
Member

Choose a reason for hiding this comment

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

@(KnownFrameworkReference) is a collection that's not specific to whatever inner or outer build is currently running. So, this is incorrect, probably something copied from one of my PRs when it wasn't completely done. The conditions were correct before modulo the $(DefaultNetCoreTargetFramework) substitution e.g. https://github.com/aspnet/EntityFrameworkCore/blob/3a7c46cdf1ffc2181e6590836157fdb5329bb68d/Directory.Build.targets#L6-L10

Please submit a fix ASAP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Issues related to our MSBuild projects and targets, build scripts, CI, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants