facebook / facebook-ios-sdk

Used to integrate the Facebook Platform with your iOS & tvOS apps.
https://developers.facebook.com/docs/ios
Other
7.8k stars 3.56k forks source link

FBSDKCoreKit.framework: Unsealed contents present in the root directory of an embedded framework #2130

Open bdaz opened 2 years ago

bdaz commented 2 years ago

Checklist before submitting a bug report

Xcode version

14.0

Facebook iOS SDK version

14.1.0

Dependency Manager

CocoaPods

SDK Framework

Core

Goals

The directory structure of FBSDKCoreKit.framework is invalid for an embedded framework, because FacebookSDKStrings.bundle is in the wrong place. This prevents macCatalyst builds when the app includes FBSDKCoreKit, because code signing will fail.

Simply moving FBSDKCoreKit/XCFrameworks/FBSDKCoreKit.xcframework/ios-arm64_x86_64-maccatalyst/FBSDKCoreKit.framework/FacebookSDKStrings.bundle to FBSDKCoreKit/XCFrameworks/FBSDKCoreKit.xcframework/ios-arm64_x86_64-maccatalyst/FBSDKCoreKit.framework/Versions/A/Resources/FacebookSDKStrings.bundle fixes this issue.

Expected results

You should be able to codesign the FBSDKCoreKit framework for macCatalyst without issue.

Actual results

/usr/bin/codesign --force --sign <abc>  --preserve-metadata=identifier,entitlements '/Users/bdaz/Library/Developer/Xcode/DerivedData/Thoughtful-cbuhkdrltepatyaqufbenzszbwvr/Build/Products/Debug-maccatalyst/Thoughtful.app/Contents/Frameworks/FBSDKCoreKit.framework'

/Users/bdaz/Library/Developer/Xcode/DerivedData/Thoughtful-cbuhkdrltepatyaqufbenzszbwvr/Build/Products/Debug-maccatalyst/Thoughtful.app/Contents/Frameworks/FBSDKCoreKit.framework: unsealed contents present in the root directory of an embedded framework

Steps to reproduce

  1. Add FBSDKCoreKit as a dependency. In my case, I'm using Cocoapods.
  2. Try to build your app for macCatalyst.

Code samples & details

No response

bdaz commented 1 year ago

This is still an issue, FYI! Really easy fix for you guys.

kesha-antonov commented 1 year ago

Thanks, it worked

boblepepeur commented 1 year ago

Please update the lib !

arshiacont commented 9 months ago

Wow! Can't believe this issue has been open for such long!!! Thanks for the workaround

andytriboletti commented 8 months ago

Still an issue. The workaround works.