Skip to content

Obsolete thumbtacked AssemblyName properties #59522

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.

Sign up for GitHub

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
jeffhandley merged 8 commits into from
Nov 16, 2021
Merged

Obsolete thumbtacked AssemblyName properties #59522

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
52cf265
Obsolete thumbtacked AssemblyName properties
i3arnon Sep 23, 2021
ec9f7ae
Ignore obsoletion
i3arnon Sep 23, 2021
486b875
Fix pragma
i3arnon Sep 23, 2021
9f05e44
Merge the AssemblyName member obsoletions into a single diagnostic id
jeffhandley Nov 15, 2021
ff6d7bc
Merge branch 'main' into assembley-name. Update to SYSLIB0037 to addr…
jeffhandley Nov 15, 2021
8455593
Fix pragma to use updated diagnostic id
jeffhandley Nov 15, 2021
b0ee59a
Suppress SYSLIB0037 in reflection tests
jeffhandley Nov 15, 2021
8d62a80
Suppress SYSLIB0037 warnings
jeffhandley Nov 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Suppress SYSLIB0037 in reflection tests
  • Loading branch information
@jeffhandley
jeffhandley committed Nov 15, 2021
commit b0ee59a99a2187d8d6641a7125183e026482f658
4 changes: 3 additions & 1 deletion src/libraries/System.Reflection.Metadata/tests/System.Reflection.Metadata.Tests.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,9 @@
<PropertyGroup>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<ExternallyShipping>false</ExternallyShipping>
<NoWarn>436</NoWarn> <!-- Type conflicts on "Interop" due to InternalsVisibleTo access -->
<!-- 436: Type conflicts on "Interop" due to InternalsVisibleTo access
SYSLIB0037: AssemblyName members HashAlgorithm, ProcessorArchitecture, and VersionCompatibility are obsolete. -->
<NoWarn>$(NoWarn);436;SYSLIB0037</NoWarn>
<TargetFrameworks>$(NetCoreAppCurrent);$(NetFrameworkMinimum)</TargetFrameworks>
<EnableDllImportGenerator>true</EnableDllImportGenerator>
</PropertyGroup>
Expand Down
5 changes: 3 additions & 2 deletions ...m.Reflection.MetadataLoadContext/tests/System.Reflection.MetadataLoadContext.Tests.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@
<PropertyGroup>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<TargetFrameworks>$(NetCoreAppCurrent);$(NetFrameworkMinimum)</TargetFrameworks>
<!-- The GAC is obsolete, but there are unit tests still referencing it -->
<NoWarn>$(NoWarn);SYSLIB0005</NoWarn>
<!-- SYSLIB0005: The GAC is obsolete, but there are unit tests still referencing it.
SYSLIB0037: AssemblyName members HashAlgorithm, ProcessorArchitecture, and VersionCompatibility are obsolete. -->
<NoWarn>$(NoWarn);SYSLIB0005;SYSLIB0037</NoWarn>
</PropertyGroup>
<ItemGroup>
<Compile Include="$(CommonTestPath)System\IO\TempDirectory.cs"
Expand Down
4 changes: 3 additions & 1 deletion src/libraries/System.Reflection/tests/System.Reflection.Tests.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
<!-- LoadFrom_SameIdentityAsAssemblyWithDifferentPath_ReturnsEqualAssemblies test relies on no deps.json -->
<GenerateDependencyFile>false</GenerateDependencyFile>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<NoWarn>$(NoWarn),SYSLIB0013</NoWarn>
<!-- SYSLIB0013: Uri.EscapeUriString is obsolete
SYSLIB0037: AssemblyName members HashAlgorithm, ProcessorArchitecture, and VersionCompatibility are obsolete. -->
<NoWarn>$(NoWarn);SYSLIB0013;SYSLIB0037</NoWarn>

<!-- these tests depend on the pdb files -->
<DebuggerSupport Condition="'$(DebuggerSupport)' == '' and '$(TargetOS)' == 'Browser'">true</DebuggerSupport>
Expand Down