Skip to content

Fix SYSLIB obsoletion warnings in tests#51847

Merged
jeffhandley merged 2 commits intodotnet:mainfrom
jeffhandley:jeffhandley/fix-39976
Apr 26, 2021
Merged

Fix SYSLIB obsoletion warnings in tests#51847
jeffhandley merged 2 commits intodotnet:mainfrom
jeffhandley:jeffhandley/fix-39976

Conversation

@jeffhandley
Copy link
Member

Fixes #39976

PR #39269 introduced warnings to the full build during .NET 5.0 when the SYSLIB obsoletions were introduced. PR #42272 configured warnings to be treated as errors and achieved a clean build, but the SYSLIB diagnostics were simply suppressed. This PR addresses the SYSLIB diagnostics through targeted changes or in-place suppressions as appropriate and removes the <NoWarn> for them.

@jeffhandley jeffhandley added test-enhancement Improvements of test source code area-Infrastructure labels Apr 26, 2021
@ghost
Copy link

ghost commented Apr 26, 2021

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #39976

PR #39269 introduced warnings to the full build during .NET 5.0 when the SYSLIB obsoletions were introduced. PR #42272 configured warnings to be treated as errors and achieved a clean build, but the SYSLIB diagnostics were simply suppressed. This PR addresses the SYSLIB diagnostics through targeted changes or in-place suppressions as appropriate and removes the <NoWarn> for them.

Author: jeffhandley
Assignees: -
Labels:

area-Infrastructure, test enhancement

Milestone: -

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.

LGTM.

@jeffhandley jeffhandley merged commit 0fc2afa into dotnet:main Apr 26, 2021
@jeffhandley jeffhandley deleted the jeffhandley/fix-39976 branch April 26, 2021 19:02
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure test-enhancement Improvements of test source code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Obsolete warnings on clean build

4 participants