[React Fabric] Updating Build Configuration (CocoaPods, .xcodeproj, gradle)

[fkgozali]

For Discussion

Various parts of React Fabric code are available in github, but we need to upgrade a bunch of build configuration to make it build properly on iOS and Android. This will involve reflecting the BUCK targets for various parts of Fabric into the 3 areas:

• React.podspec (iOS) -- note that we had partial configs already, but very outdated.
• React.xcodeproj (iOS)
• build.gradle (Android)

An initial version of the update could just be manually sync'ed configs for .podspec, .xcodeproj, and .gradle.

But for the future, some ideas include:

• Have BUCK generate the relevant .podspec, .xcworkspace, or .gradle -- this may require a bunch of work via buck project
• etc

Requirements

• Have RNTester built with Fabric symbols correctly (iOS/Android to start)
• Upgraded version of Folly - see [META] Upgrading Folly #20302
  • Note that we're on v2018.10.22 already (Upgrade folly to v2018.10.22.00 for iOS #21976, Upgrade folly to v2018.10.22.00 for Android #21977)

cc @mdvacca, @shergin

[danibonilha]

Hey there! I'd like to help, so I started with the podspec, and I'd appreciate any tips, on how to test and make sure everything is working. Thanks.

[fkgozali]

on how to test and make sure everything is working

First step is to have RNTester podspec able to build Fabric specs (that includes all subspecs from fabric directory).

The next step after that is to actually use Fabric in RNTester. This is still work in progress for our team, so we can tackle that slightly later. cc @shergin

[danibonilha]

Hey, could you guys please verify if this include is correct? I'm getting an error 'Unknown type name 'CGFloat'' in this file and looking at the code '<CoreGraphics/CoreGraphics.h>' is always used with .m files and import instead of include and .cpp files.

[fkgozali]

Hey, could you guys please verify if this include is correct? I'm getting an error 'Unknown type name 'CGFloat'' in this file and looking at the code '<CoreGraphics/CoreGraphics.h>' is always used with .m files and import instead of include and .cpp files.

Does it work if you modify it to #import ? This builds fine using Buck at FB, perhaps there's a missing flag somewhere? Or perhaps you're missing the dependency on CoreGraphics iOS framework in the podspec?

If moving to #import works feel free to send a PR for it. Looking at a similar file, seems like we use #import there:

react-native/ReactCommon/fabric/imagemanager/platform/ios/RCTImagePrimitivesConversions.h

Line 8 in 94d49e5

#import <UIKit/UIKit.h>

[grabbou]

@fkgozali are there any question marks you see as far as using CocoaPods with Fabric/TurboModules? I'd love to hear more about your experience so far.

[mgriepentrog]

I can take a look at updating the xcodeproj files.

[fkgozali]

are there any question marks you see as far as using CocoaPods with Fabric/TurboModules? I'd love to hear more about your experience so far.

We don't use cocoapods/gradle internally, so I'm hoping the unknowns can be covered with the help from community on this issue.

[danibonilha]

Hey React.podspec is up for grabs again, I'm sorry, I didn't have enough time to finish it up.

[mgriepentrog]

I switched gears to updating the podspec (xcodeprojs are a pain). Here is what I have so far: mgriepentrog@44c5649. To find a lot of the low-hanging fruit, I'm running pod install inside of RNTester and trying to build the app with the RCTFabric subspec included. Currently encountering react-native/RNTester/Pods/Headers/Private/React/react/graphics/Geometry.h:55:3: Unknown type name 'Float'; did you mean 'float'?, but haven't had much time to investigate.

[danibonilha]

I switched gears to updating the podspec (xcodeprojs are a pain). Here is what I have so far: mgriepentrog@44c5649. To find a lot of the low-hanging fruit, I'm running pod install inside of RNTester and trying to build the app with the RCTFabric subspec included. Currently encountering react-native/RNTester/Pods/Headers/Private/React/react/graphics/Geometry.h:55:3: Unknown type name 'Float'; did you mean 'float'?, but haven't had much time to investigate.

I got stuck on the same problem, and couldn't get pass it I added same frameworks, changed some files and then same problem again. 😟

[fkgozali]

It looks like this header file wasn't included in your setup?

react-native/ReactCommon/fabric/graphics/platform/ios/Float.h

Line 18 in 3ad5c9e

using Float = CGFloat;

[ericlewis]

I made progress on getting the cocoapods to build and ran in to a few problems:

• CGFloat in Float.h, same problems as above, it seems that the interop isn't working, I stubbed this with a double to continue.
• RCTSwitchComponentView makes references to react/components/rncore/, which I could not find, nor could I find something similar.
• Missing x86_64 Symbol Error on getDefaultComponentRegistryFactory, but that might be due to my stubbing? <- This is due to missing folly imports.

I am still working on getting it to build a little further, and and still trying to figure out what is going on with Float.h.

But the podfile seems mostly complete: ericlewis@a6510d9. It creates libReact.a.