iOS build fails with "Use of undeclared identifier 'RNNordicDfu'" error

[mtsap]

Tried it with this repo as well the one mentioned in the Readme.

Also tried adding the following line to the Podfile
pod "react-native-nordic-dfu", path: "../node_modules/react-native-nordic-dfu"
with no success.

npx react-native info output:

System: OS: macOS 12.4 CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz Memory: 29.89 MB / 8.00 GB Shell: 5.9 - /usr/local/bin/zsh Binaries: Node: 12.19.0 - ~/.nvm/versions/node/v12.19.0/bin/node Yarn: 1.22.19 - /usr/local/bin/yarn npm: 6.14.15 - ~/node-projects/arion2-app/node_modules/.bin/npm Watchman: 2022.06.13.00 - /usr/local/bin/watchman Managers: CocoaPods: 1.11.2 - /usr/local/bin/pod SDKs: iOS SDK: Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5 Android SDK: Android NDK: 22.1.7171670 IDEs: Android Studio: 4.2 AI-202.7660.26.42.7486908 Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild Languages: Java: 10.0.2 - /usr/bin/javac npmPackages: @react-native-community/cli: Not Found react: 17.0.1 => 17.0.1 react-native: 0.64.3 => 0.64.3 react-native-macos: Not Found npmGlobalPackages: *react-native*: Not Found

[tale-dev]

@mtsap it's properly because of node version, you can try node v14.16.1

[samuel-gallet]

Does not build either with node v14.16.1 @tale-dev

[tale-dev]

@samuel-gallet thanks, are there any problem with node v14.16.1?

[mtsap]

@samuel-gallet @tale-dev for me it buit. It was my mistake. I put the imports below this line:
#ifdef FB_SONARKIT_ENABLED

Moved them above and now it works.

[samuel-gallet]

I made the same mistake too @mtsap .

However I could not make it build because of a "modules are disabled" error.
I get Use of '@import' when C++ modules are disabled, consider using -fmodules and -fcxx-modules error in the RNNordicDfy.h file

[CptFabulouso]

Solution 1)
I solved it by adding the -fcxx-modules to Other C++ Flags (in Build Settings) with RN 0.70.6. I couldn't make it work after manually upgrading from RN 0.66.1, there was some different error, I had to first create a new clean RN project, copy content of ios folder to my project and fix changes in app icons, font resources etc. But that was just my case. Note that I think you can't use the new RN architecture with -fcxx-modules flag.

Solution 2)
To make it work without that flag the solution is following:

• replace the @import with #import <iOSDFULibrary/iOSDFULibrary-Swift.h> in RNNordicDfu.h. Patch below
• add use_frameworks! to your Podfile
  • Flipper doesn't work with this, that's why I didn't choose this option
  • If you have hermes enabled, use use_frameworks! :linkage => :static use_framework! + hermes enable = Undefined symbols "vtable for facebook::react::HermesExecutorFactory" facebook/react-native#34344 (comment)
  • Not sure what other things/libs it may break

Patch

diff --git a/node_modules/react-native-nordic-dfu/ios/RNNordicDfu.h b/node_modules/react-native-nordic-dfu/ios/RNNordicDfu.h
index 619a95b..8632a49 100644
--- a/node_modules/react-native-nordic-dfu/ios/RNNordicDfu.h
+++ b/node_modules/react-native-nordic-dfu/ios/RNNordicDfu.h
@@ -1,7 +1,7 @@
 #import <CoreBluetooth/CoreBluetooth.h>
 #import <React/RCTBridgeModule.h>
 #import <React/RCTEventEmitter.h>
-@import iOSDFULibrary;
+#import <iOSDFULibrary/iOSDFULibrary-Swift.h>
 
 @interface RNNordicDfu : RCTEventEmitter<RCTBridgeModule, DFUServiceDelegate, DFUProgressDelegate, LoggerDelegate>

[manualexSP]

Hi @CptFabulouso thank you for the explanation of the import problem. I will add a fix to this fork and I will update the readme.

[CptFabulouso]

Ok. Btw I now think the second solution should also work with Flipper. I managed to make work Flipper with newest react-native-firebase, which also needs to use use_frameworks.

My solution was to add following to Podfile:

# items in array are for Firebase and should be changed to include the DFU lib, not sure how exactly ATM.
static_frameworks = ['Firebase','FirebaseCoreInternal', 'FirebaseCoreExtension', 'FirebaseInstallations', 'FirebaseCore', 'FirebaseMessaging', 'GoogleUtilities']  
pre_install do |installer|
  installer.pod_targets.each do |pod|
    if static_frameworks.include?(pod.name)
      puts "Overriding the static_frameworks? method for #{pod.name}"
      def pod.build_type;
        Pod::BuildType.new(:linkage => :static, :packaging => :framework)
      end
    end
  end
end

target 'YourApp' do
    ...
end

And if that does not work, someone found another approach described here.
So if you can take the time to find out some of these approaches work, you may also mention that in the readme.

[ervibern]

This is really promising. Currently struggling to get it to build for an Expo app. I used @manualexSP fork with the readme instructions but it didn't build. If anyone could help, would really appreciate it. If I don't want to lose Flipper functionality, should I try static_frameworks = ['react-native-nordic-dfu'] as per @CptFabulouso suggestion?

Podfile

target 'MyApp' do
  use_expo_modules!
  config = use_native_modules!
  
  // added those 2 lines
  pod "react-native-nordic-dfu", path: "../node_modules/react-native-nordic-dfu"
  use_frameworks! :linkage => :static

  use_frameworks! :linkage => podfile_properties['ios.useFrameworks'].to_sym if podfile_properties['ios.useFrameworks']

  # Flags change depending on the env values.
  flags = get_default_flags()

  use_react_native!(
    :path => config[:reactNativePath],
# @generated begin expo-community-flipper-isprod - expo prebuild (DO NOT MODIFY) sync-828c22a1a38236bf5b7c203393f474bc68356b34
    # ENV value added to support Hermes
    :production => ENV["PRODUCTION"] == "1" ? true : false,
# @generated end expo-community-flipper-isprod
    :hermes_enabled => podfile_properties['expo.jsEngine'] == 'hermes',
    :fabric_enabled => flags[:fabric_enabled],
# @generated begin expo-community-flipper-urn - expo prebuild (DO NOT MODIFY) sync-c7cb57ad0548737963234cf3378d4b0e06dca78b
    :flipper_configuration => FlipperConfiguration.enabled(["Debug", "Release"]),
# @generated end expo-community-flipper-urn
    # An absolute path to your application root.
    :app_path => "#{Pod::Config.instance.installation_root}/..",
    #
    # Uncomment to opt-in to using Flipper
    # Note that if you have use_frameworks! enabled, Flipper will not work
    # :flipper_configuration => !ENV['CI'] ? FlipperConfiguration.enabled : FlipperConfiguration.disabled,
  )

  post_install do |installer|
    react_native_post_install(
      installer,
      # Set `mac_catalyst_enabled` to `true` in order to apply patches
      # necessary for Mac Catalyst builds
      :mac_catalyst_enabled => false
    )
    __apply_Xcode_12_5_M1_post_install_workaround(installer)

    # This is necessary for Xcode 14, because it signs resource bundles by default
    # when building for devices.
    installer.target_installation_results.pod_target_installation_results
      .each do |pod_name, target_installation_result|
      target_installation_result.resource_bundle_targets.each do |resource_bundle_target|
        resource_bundle_target.build_configurations.each do |config|
          config.build_settings['CODE_SIGNING_ALLOWED'] = 'NO'
        end
      end
    end
  end

  post_integrate do |installer|
    begin
      expo_patch_react_imports!(installer)
    rescue => e
      Pod::UI.warn e
    end
  end
end

AppDelegate.mm

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  RCTAppSetupPrepareApp(application);

  RCTBridge *bridge = [self.reactDelegate createBridgeWithDelegate:self launchOptions:launchOptions];

#if RCT_NEW_ARCH_ENABLED
  _contextContainer = std::make_shared<facebook::react::ContextContainer const>();
  _reactNativeConfig = std::make_shared<facebook::react::EmptyReactNativeConfig const>();
  _contextContainer->insert("ReactNativeConfig", _reactNativeConfig);
  _bridgeAdapter = [[RCTSurfacePresenterBridgeAdapter alloc] initWithBridge:bridge contextContainer:_contextContainer];
  bridge.surfacePresenter = _bridgeAdapter.surfacePresenter;
#endif

  NSDictionary *initProps = [self prepareInitialProps];
  UIView *rootView = [self.reactDelegate createRootViewWithBridge:bridge moduleName:@"main" initialProperties:initProps];

  rootView.backgroundColor = [UIColor whiteColor];
  self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];
  UIViewController *rootViewController = [self.reactDelegate createRootViewController];
  rootViewController.view = rootView;
  self.window.rootViewController = rootViewController;
  [self.window makeKeyAndVisible];

  [super application:application didFinishLaunchingWithOptions:launchOptions];
  
  // Added those 3 blocks
  [RNNordicDfu setCentralManagerGetter:^() {
        return [[CBCentralManager alloc] initWithDelegate:nil 
        queue:dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_BACKGROUND, 0)];
   }];

  [RNNordicDfu setOnDFUComplete:^() {
        NSLog(@"onDFUComplete");
   }];
    
  [RNNordicDfu setOnDFUError:^() {
        NSLog(@"onDFUError");
   }];

  return YES;
}

[manualexSP]

@ervibern I will give it a try to the @CptFabulouso mention and I will come back this week to you guys.

[manualexSP]

Ok so I tested on a react-native 0.71.8 project, the pre_install logic with FlipperConfiguration.enabled and it works.

  static_frameworks = ['iOSDFULibrary']  
  pre_install do |installer|
    installer.pod_targets.each do |pod|
      if static_frameworks.include?(pod.name)
        puts "Overriding the static_frameworks? method for #{pod.name}"
        def pod.build_type;
          Pod::BuildType.new(:linkage => :static, :packaging => :framework)
        end
      end
    end
  end

I updated the README and I put two solutions one with FlipperConfiguration.disabled and one with FlipperConfiguration.enabled. check here

I also added a new section with Issues and I put a link from this conversation in case others have problems too.

@ervibern give it a try now and let us know how it works. Also, I never tested this library with Expo and I never worked with it, so I'm not sure how things are going to work for you.

[ervibern]

@manualexSP Can confirm that your current fork builds and works on my app with react native 0.70.8 and expo 47.0.14! :) Flipper is enabled and works together with DFU package. I have to figure out how to add those custom modifications to Podfile and AppDelegate.mm to the Expo project but this is a separate topic. Thanks for your work!