Skip to content

[release/6.0] Suppress warnings for deprecated keychain APIs #77926

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

Conversation

vcsjones
Copy link
Member

@vcsjones vcsjones commented Nov 4, 2022

Backport of #77911 to release/6.0

/cc @vcsjones

Customer Impact

This change fixes compiling with the latest Xcode tooling.

In Xcode 14.1, Apple deprecated several APIs related to their Keychain APIs. This change suppresses the obsolete warnings so the build continues. The Mac Pools for CI will need this change for CI to continue to build when the Mac Pool eventually gets updated to this version of the tooling.

Testing

Verified compilation succeeds with Xcode 14.1 toolchain.

Risk

Very low. This makes no functional changes, and fixes a build-break using #pragma directives to suppress the warning.

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

@vcsjones vcsjones added this to the 6.0.x milestone Nov 4, 2022
@ghost ghost assigned vcsjones Nov 4, 2022
@ghost
Copy link

ghost commented Nov 4, 2022

Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #77911 to release/6.0

/cc @vcsjones

Customer Impact

This change fixes compiling with the latest Xcode tooling.

In Xcode 14.1, Apple deprecated several APIs related to their Keychain APIs. This change suppresses the obsolete warnings so the build continues. The Mac Pools for CI will need this change for CI to continue to build when the Mac Pool eventually gets updated to this version of the tooling.

Testing

Verified compilation succeeds with Xcode 14.1 toolchain.

Risk

Very low. This makes no functional changes, and fixes a build-break using #pragma directives to suppress the warning.

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author: vcsjones
Assignees: -
Labels:

area-System.Security

Milestone: 6.0.x

@vcsjones vcsjones added the os-mac-os-x macOS aka OSX label Nov 4, 2022
@akoeplinger akoeplinger added the Servicing-consider Issue for next servicing release review label Nov 7, 2022
@akoeplinger
Copy link
Member

This is a build-only change

@carlossanlop
Copy link
Contributor

@vcsjones or @akoeplinger can you please send the email to Tactics to request merge approval?

@vcsjones
Copy link
Member Author

vcsjones commented Nov 9, 2022

@bartonjs @akoeplinger I lack this capability / knowledge.

@ericstj
Copy link
Member

ericstj commented Nov 10, 2022

These are just pragmas, so we can treat it as an infrastructure change and just make it "tell mode" to tactics. Let's just draft an email that let's them know we're merging this (and the 7.0 PR) in order to be able to continue building this component with the latest toolchain.

@akoeplinger
Copy link
Member

Too late, I've already sent the tactics email a few minutes ago 😄

@carlossanlop carlossanlop added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 10, 2022
@carlossanlop
Copy link
Contributor

Ok got it. Tell mode, so I'll add the label. Thank you Alexander for sending the email anyway.
CI is green. Signed off by area owner. No OOB package authoring changes needed. Ready to merge. :shipit:

@carlossanlop carlossanlop merged commit fe985d9 into dotnet:release/6.0 Nov 10, 2022
@vcsjones vcsjones deleted the backport/pr-77911-to-release/6.0 branch November 10, 2022 19:26
@ghost ghost locked as resolved and limited conversation to collaborators Dec 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Security os-mac-os-x macOS aka OSX Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants