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

building fails on mac #30

Closed
ouwou opened this issue Jan 22, 2023 · 1 comment · Fixed by #31
Closed

building fails on mac #30

ouwou opened this issue Jan 22, 2023 · 1 comment · Fixed by #31

Comments

@ouwou
Copy link
Contributor

ouwou commented Jan 22, 2023

cant really test myself, but github actions gave me this:

  Error: /Users/runner/work/abaddon/abaddon/subprojects/keychain/src/keychain_mac.cpp:115:23: error: 'SecKeychainFindGenericPassword' is deprecated: first deprecated in macOS 10.10 - SecKeychain is deprecated [-Werror,-Wdeprecated-declarations]
      OSStatus status = SecKeychainFindGenericPassword(
                        ^
  /Applications/Xcode_14.2.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.1.sdk/System/Library/Frameworks/Security.framework/Headers/SecKeychain.h:628:10: note: 'SecKeychainFindGenericPassword' has been explicitly marked deprecated here
  OSStatus SecKeychainFindGenericPassword(CFTypeRef __nullable keychainOrArray,  UInt32 serviceNameLength, const char * __nullable serviceName, UInt32 accountNameLength, const char * __nullable accountName, UInt32 * __nullable passwordLength, void * __nullable * __nullable passwordData, SecKeychainItemRef * __nullable CF_RETURNS_RETAINED itemRef)
           ^

maybe a -Wno-error=deprecated-declarations would suffice?

$<$<NOT:$<CXX_COMPILER_ID:MSVC>>:-Wall -Wextra -pedantic -Werror>)

hrantzsch added a commit that referenced this issue Jan 22, 2023
We're using the deprecated "Legacy Password Storage" API on macOS.

Fixes #30
@hrantzsch
Copy link
Owner

Indeed, thanks for letting me know. Unfortunately the API we're using is deprecated. As far as I know there's no replacement available that doesn't require Xcode, or at least some involved setup of App capabilities.

hrantzsch added a commit that referenced this issue Jan 22, 2023
We're using the deprecated "Legacy Password Storage" API on macOS.

Fixes #30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants