Summary:
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1268
Not sure why we were weak-linking core location. I am assuming this was related in some way to the deprecated PlacesKit but am not positive.
Reviewed By: Oliverccccct
Differential Revision: D20167000
fbshipit-source-id: 4b8e69d3e263667cb29ac6ecebb87c0da3ba93bb
Summary:
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1220
Bumps sdk version to 6.0.0
Bumps api version to v6.0
Reviewed By: jingping2015, ZebingZong, KylinChang
Differential Revision: D19698716
fbshipit-source-id: 165ad3f3ac0f5c686e965f521e9d807f9f21d523
Summary:
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
Updates Package file and Podspecs to rely on file-level platform checks.
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1186
Test Plan:
Sample project using this branch should build for iOS, tvOS, Mac Catalyst
Also checked size implications. This change causes a 16 byte increase in size which seems like more than a fair trade-off for the gains in simplicity.
Reviewed By: ZebingZong
Differential Revision: D19499591
Pulled By: joesus
fbshipit-source-id: 50122a6cbeca0e24f9813cde1907affd9dbfa8c4
Summary:
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
What we have is similar to this issue here. https://github.com/CocoaPods/CocoaPods/issues/9101
Basically we need to expose FBSDKCOCOAPODS for static libs to work with other kits. There may be other convoluted workarounds but I'd argue it's worth taking the chance that few people are
using the SDK from CocoaPods as a static library AND are unable to avoid a macro collision with `FBSDKCOCOAPODS`
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1184
Test Plan: `pod lib lint FBSDKLoginKit.podspec --allow-warnings --use-libraries --include-podspecs=FBSDKCoreKit.podspec`
Reviewed By: Mxiim
Differential Revision: D19425678
Pulled By: joesus
fbshipit-source-id: 756121ee8cc90a5d9fc2a8b7e7e5ace3b8613394
Summary:
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
Should be able to declare `FBSDKCOCOAPODS` more granularly.
See: https://github.com/CocoaPods/CocoaPods/issues/3309
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1180
Test Plan: N/A
Reviewed By: ZebingZong
Differential Revision: D19386668
Pulled By: joesus
fbshipit-source-id: 043bd715aadf466ac8403543ca6fea43827fda45
Summary:
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1177
D19247723 moved relative files to a new folder named GraphAPI but haven't updated the requires_arc setting for this folder, so add back requires_arc setting back to avoid crash on sdk launch.
Reviewed By: joesus
Differential Revision: D19357327
fbshipit-source-id: 21e3c2577d2a26b5386d76a42fa283078d75adcb
Summary:
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1173
When I tried to integrate FBSDK with cocoapods, getting undefined symbol errors in files of `FBSDKEventInferencer.mm` and `FBSDKAddressInferencer.mm`. This is because FBSDKCoreKit.podspec hasn't specified c++ library to compile.
{F226154380}
Reviewed By: joesus
Differential Revision: D19326280
fbshipit-source-id: a070fc22a56e245f6c7193f13f5adfd34c88c624
Summary:
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1139
- move ViewHierarchy.{h,m} and macros to folder ViewHierarchy
- delete unused FBSDKFunctionEventBinding.h in MarketingKit. The interface FBSDKFunctionEventBinding doesn't have the implementaion but just a .h.
Reviewed By: joesus
Differential Revision: D18674274
fbshipit-source-id: 24c555a2adf0ae04899ffce4a2c73edec688b759
Summary:
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
Fixing import errors when using Unity.
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1122
Test Plan: Open a new unity project that includes the SDK. Open the generated Xcode workspace. It should compile without error about improper header import syntax.
Reviewed By: Mxiim
Differential Revision: D18572442
Pulled By: joesus
fbshipit-source-id: bddea9a5b97d18f8b27b583dfcce6866c382aea0
Summary:
as title
Following diffs will include:
* add old version file clear function
* for loop to enable each feature/usecase
Reviewed By: dreamolight
Differential Revision: D18362851
fbshipit-source-id: 322823fbfe9308f8522d44768aa78d19aae234e7
Summary:
Cocoapods will pass allow directories full of symlinks during linting but not while pushing to trunk.
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
This is very strange. Cocoapods was able to lint these podspecs successfully but pushing them to trunk resulted in an error.
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1098
Test Plan: Ran manually and pods are now available in trunk.
Reviewed By: dreamolight
Differential Revision: D18383388
Pulled By: joesus
fbshipit-source-id: 3012c3a1128531464aedac1d17094116c1767a32
Summary: - fetch feature GK from server to control enabling suggested events
Reviewed By: KylinChang
Differential Revision: D18383293
fbshipit-source-id: 67d40381d8e351da1764540e1e7bd5ef52037713
Summary:
Thanks for proposing a pull request!
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] I've ensured that all existing tests pass and added tests (when/where necessary)
- [x] I've updated the documentation (when/where necessary) and [Changelog](CHANGELOG.md) (when/where necessary)
- [x] I've added the proper label to this pull request (e.g. `bug` for bug fixes)
## Pull Request Details
Added a `Package.swift` manifest.
Updated imports to be non-modular for non-cocoapod setups.
Updated dynamic framework loader to compile with arc. - This should be checked. Not 100% sure I did not cause a regression.
Added a directory of symlinks to public header files - this was to get around a limitation in the SPM api where they will only accepts a path to header files (forces them to be located in the same directory) and not a list of header files.
Pull Request resolved: https://github.com/facebook/facebook-ios-sdk/pull/1073
Test Plan:
**SPM:**
`cd fbsource/fbobjc/ios-sdk`
Initialize a new repository, add all the files and create a new commit on master.
`git init`
`git add .`
`git commit -m "new commit"`
Open the package itself with >= Xcode 11 by opening `Package.swift` instead of the usual project and verify that it builds
Open a new project with >= Xcode 11 and add a new swift package. For the Package Repository field put the path to the sdk.
`file:///Users/$whoami/fbsource/fbobjc/ios-sdk`
then specify the branch as `master`. This sample project should build and you should be able to consume the kits with `import FBSDKCoreKit` or `FacebookCore` (for the Swift variant), `import FBSDKLoginKit` or `import FacebookLogin` and `import FBSDKShareKit` or `import FacebookShare`.
Make sure that LoginKit and ShareKit also expose CoreKit symbols.
**Xcode:**
Can build all schemes as expected
`BuildAllKits`
`FBSDKTVOSKit-Universal`
`FBSDKTVOSKit-Dynamic`
`SwiftKits`
**CocoaPods:**
`cd fbsource/fbojbc/ios-sdk`
`bundle exec pod lib lint FBSDKCoreKit.podspec`
`bundle exec pod lib lint FBSDKLoginKit.podspec --include-podspecs=FBSDKCoreKit.podspec --allow-warnings`
`bundle exec pod lib lint FBSDKShareKit.podspec --include-podspecs=FBSDKCoreKit.podspec --allow-warnings`
**BUCK:**
`cd fbsource/fbobjc/ios-sdk`
`buck build ios-sdk:FBSDKCoreKit`
`buck build ios-sdk:FBSDKLoginKit`
`buck build ios-sdk:FBSDKShareKit`
Also all of the other BUCK targets (will be covered by sandcastle)
Reviewed By: dreamolight
Differential Revision: D17879652
Pulled By: joesus
fbshipit-source-id: d931140d0fb3b554f1e29e5c2af7e2173316670d
Summary: The GitHub repo has been renamed from facebook-ios-sdk to facebook-objc-sdk
Reviewed By: jingping2015
Differential Revision: D17630050
fbshipit-source-id: 4fa7a3c4859af7fead71c4700a1151903d4734cc
Summary: Add AAM to FBSDKCoreKit along with changes in podspec and BUCK
Reviewed By: tianqibt
Differential Revision: D17532337
fbshipit-source-id: 770f421602ba8757dce4db3cebf4091605e0169d
Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] describe the change (for example, what happens before the change, and after the change)
This is a step towards aligning Swift and Objective-C into a single SDK.
Currently we cannot ship a single target that includes both Swift and Objective-C because some consumers of the SDK are tied to older versions of Xcode. Including Swift would increase their binary size or force them to upgrade before Apple mandates an upgrade.
What we can do is prepare for this inevitability by moving the Swift files into this repository.
Additionally this is easier to maintain and allows us to us to reference ObjC files from Swift directly instead of treating them as a third party dependency.
This work raises a few questions about distribution channels (what will the starting versions be for instance) but that is an acceptable cost for the benefits we get from having all of our related code in one place.
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/1031
Test Plan:
Able to build is the primary concern. That the old targets are not impacted by the new targets and that the new targets build correctly.
Ensure the older targets can build on Xcode 10.1
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
Ensure the older AND newer targets can build on Xcode 10.3
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
* FBSDKCoreKitSwift
* FBSDKShareKitSwift
* FBSDKLoginKitSwift
Ensure the older AND newer targets can build on the latest Xcode (11 beta 6)
// Note: Will need to clean build if you want to build the non-Swift version after building the Swift version
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
* FBSDKCoreKitSwift
* FBSDKShareKitSwift
* FBSDKLoginKitSwift
Reviewed By: dreamolight
Differential Revision: D17164409
Pulled By: joesus
fbshipit-source-id: dbda5036487a2e626e271651b38262e56dac17ce
