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/7.0] Suppress warnings for deprecated keychain APIs #77924

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 4, 2022

Backport of #77911 to release/7.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.

@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/7.0

/cc @vcsjones

Customer Impact

Testing

Risk

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: github-actions[bot]
Assignees: -
Labels:

area-System.Security

Milestone: -

@vcsjones vcsjones added the os-mac-os-x macOS aka OSX label Nov 4, 2022
@vcsjones vcsjones added this to the 7.0.x milestone Nov 4, 2022
@marek-safar marek-safar added the Servicing-consider Issue for next servicing release review label Nov 5, 2022
@marek-safar
Copy link
Contributor

This is a build change only

@carlossanlop
Copy link
Member

CI failures unrelated: #77924 and #77988
Tactics approved. Signed off by area owner. No OOB package authoring changes needed.
Ready to merge. :shipit:

@carlossanlop carlossanlop merged commit 3721bc8 into release/7.0 Nov 10, 2022
@carlossanlop carlossanlop deleted the backport/pr-77911-to-release/7.0 branch November 10, 2022 22:33
@carlossanlop carlossanlop modified the milestones: 7.0.x, 7.0.1 Nov 11, 2022
@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