Fix compilation for deprecated API on macOS Mojave preview (#30716)

* Fix compilation for deprecated API on macOS Mojave preview Fixes: #30599 * Fixing tests on macOS Mojave
dotnet · Jun 28, 2018 · 5e46fe1 · 5e46fe1
1 parent 50ba6a2
commit 5e46fe1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/Native/Unix/System.Security.Cryptography.Native.Apple/CMakeLists.txt b/src/Native/Unix/System.Security.Cryptography.Native.Apple/CMakeLists.txt
@@ -25,6 +25,9 @@ set(NATIVECRYPTO_SOURCES
     pal_x509chain.c
 )
 
+# Temporary workaround for dotnet/corefx issue #30599
+add_compile_options(-Wno-deprecated-declarations)
+
 add_library(System.Security.Cryptography.Native.Apple
     SHARED
     ${NATIVECRYPTO_SOURCES}

diff --git a/src/Native/Unix/System.Security.Cryptography.Native.Apple/pal_x509chain.c b/src/Native/Unix/System.Security.Cryptography.Native.Apple/pal_x509chain.c
@@ -186,7 +186,7 @@ static void MergeStatusCodes(CFTypeRef key, CFTypeRef value, void* context)
         // It doesn't represent a new error code, and we're still getting the old ones, so
         // just ignore it for now.
     }
-    else if (CFEqual(keyString, CFSTR("NonEmptySubject")))
+    else if (CFEqual(keyString, CFSTR("NonEmptySubject")) || CFEqual(keyString, CFSTR("GrayListedKey")))
     {
         // Not a "problem" that we report.
     }