Skip to content

Include Dependency Frameworks in set of TFMs considered for package testing #7645

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

Merged
merged 2 commits into from
Jul 20, 2021

Conversation

ericstj
Copy link
Member

@ericstj ericstj commented Jul 19, 2021

Fixes dotnet/runtime#27503

I believe this should fix testing for meta-packages as it will consider Frameworks for testing if they are used in dependency groups (not just files).

I haven't been able to try this out in dotnet/runtime yet and would want to do so before merging.

@ericstj ericstj requested review from ViktorHofer, Anipik and safern July 19, 2021 21:23
@ericstj
Copy link
Member Author

ericstj commented Jul 19, 2021

Tested locally and this does catch the windows compat bug:

C:\src\dotnet\runtime\artifacts\bin\testPackages\projects\Microsoft.Windows.Compatibility\project.csproj : error NU1603: Microsoft.Windows.Compatibility 6.0.0-dev depends on System.Data.SqlClient (>= 4.8.2-dev) but System.Data.SqlClient 4.8.2-dev was not found. An approximate best match of System.Data.SqlClient 4.8.2 was resolved. [C:\src\dotnet\runtime\artifacts\bin\testPackages\dirs.proj]

Also found a new failure:

C:\src\dotnet\runtime\artifacts\bin\testPackages\projects\Microsoft.Windows.Compatibility\project.csproj : error NU1603: runtime.native.System.IO.Ports 6.0.0-dev depends on runtime.linux-arm.runtime.native.System.IO.Ports (>= 6.0.0-dev) but runtime.linux-arm.runtime.native.System.IO.Ports 6.0.0-dev was not found. An approximate best match of runtime.linux-arm.runtime.native.System.IO.Ports 6.0.0-preview.1.21071.15 was resolved. [C:\src\dotnet\runtime\artifacts\bin\testPackages\dirs.proj]
C:\src\dotnet\runtime\artifacts\bin\testPackages\projects\Microsoft.Windows.Compatibility\project.csproj : error NU1603: runtime.native.System.IO.Ports 6.0.0-dev depends on runtime.linux-arm64.runtime.native.System.IO.Ports (>= 6.0.0-dev) but runtime.linux-arm64.runtime.native.System.IO.Ports 6.0.0-dev was not found. An approximate best match of runtime.linux-arm64.runtime.native.System.IO.Ports 6.0.0-preview.1.21071.15 was resolved. [C:\src\dotnet\runtime\artifacts\bin\testPackages\dirs.proj]
C:\src\dotnet\runtime\artifacts\bin\testPackages\projects\Microsoft.Windows.Compatibility\project.csproj : error NU1603: runtime.native.System.IO.Ports 6.0.0-dev depends on runtime.linux-x64.runtime.native.System.IO.Ports (>= 6.0.0-dev) but runtime.linux-x64.runtime.native.System.IO.Ports 6.0.0-dev was not found. An approximate best match of runtime.linux-x64.runtime.native.System.IO.Ports 6.0.0-preview.1.21071.15 was resolved. [C:\src\dotnet\runtime\artifacts\bin\testPackages\dirs.proj]
C:\src\dotnet\runtime\artifacts\bin\testPackages\projects\Microsoft.Windows.Compatibility\project.csproj : error NU1603: runtime.native.System.IO.Ports 6.0.0-dev depends on runtime.osx-x64.runtime.native.System.IO.Ports (>= 6.0.0-dev) but runtime.osx-x64.runtime.native.System.IO.Ports 6.0.0-dev was not found. An approximate best match of runtime.osx-x64.runtime.native.System.IO.Ports 6.0.0-preview.1.21071.15 was resolved. [C:\src\dotnet\runtime\artifacts\bin\testPackages\dirs.proj]

Here we'll just need to disable that project since it's expected that a single build isn't producing all these packages.

Copy link
Contributor

@Anipik Anipik left a comment

Choose a reason for hiding this comment

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

LGTM, just need a test in place.

Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

Thanks!

@Anipik Anipik merged commit d314438 into dotnet:main Jul 20, 2021
@ericstj
Copy link
Member Author

ericstj commented Jul 20, 2021

Thanks @Anipik. I merged dotnet/runtime#55976 so we should be ready to accept this change. It's still possible that new failures are introduced. @missymessa let us know if you see any dotnet/runtime failures around package testing during arcade ingestion.

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.

LGTM

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.

Package testing skips Microsoft.Windows.Compatibility
4 participants