Error "no implicit conversion of String into Array" in Pod Install after migrating to RN > 0.74 (0.74.0 until 0.75.3) #46505

MasGaNo · 2024-09-16T08:30:08Z

Description

I'm working on a very old application (RN 0.59.3) and we tried to keep it in shape by upgrading all dependencies, especially on RN version. So we might have many legacy configurations we bring from a version to another one, even if we are doing our best to clean as much as possible.

After migrating from RN 0.73.9 to any version > 0.74 including the latest 0.75.3 version, an error occurs during the pod installation:

cd ios && pod install

With the following error: no implicit conversion of String into Array

After some investigation, it appears that in the /node_modules/react-native/scripts/cocoapods/utils.rb file, the new method add_flag_to_map_with_inheritance was introduced and deals with an array of string settings value.

For some reason, on the method above, this method is called for OTHER_CPLUSPLUSFLAGS and OTHER_CFLAGS settings.
If I tried to output these values, it appears they contain Array instead of string value such as: ["$(inherited)", "-DNDEBUG"]

And so obviously it fails, as the method tries to concat a string with an array

As a quick workaround, if I replace the following lines:

            unless map[field].include?(flag)
                map[field] = map[field] + flag
            end
            unless map[field].include?("$(inherited)")
                map[field] = "$(inherited) " + map[field]
            end

By:

            unless map[field].include?(flag)
                if map[field].instance_of? String
                    map[field] = map[field] + flag
                elsif map[field].instance_of? Array
                    map[field].push(flag)
                end
            end
            unless map[field].include?("$(inherited)")
                if map[field].instance_of? String
                    map[field] = "$(inherited) " + map[field]
                elsif map[field].instance_of? Array
                    map[field].unshift("$(inherited)")
                end
            end

in order to handle both Array and String, it works like a charm and I can complete the pod installation and even fully build my iOS App.

But I'm not sure it something you expect and I'm not able to find where the config.build_settings is filled from:

                self.add_flag_to_map_with_inheritance(config.build_settings, 'OTHER_CPLUSPLUSFLAGS', ndebug_flag);
                self.add_flag_to_map_with_inheritance(config.build_settings, 'OTHER_CFLAGS', ndebug_flag);

in order to understand why I'm getting an array of string instead of a string.

Any hints?

Steps to reproduce

Using an old application < 0.74
Bumping RN version to > 0.74
cd ios && pod install

React Native Version

0.75.3

Affected Platforms

Runtime - iOS, Build - MacOS

Output of `npx react-native info`

System:
  OS: macOS 13.5.2
  CPU: (12) arm64 Apple M2 Pro
  Memory: 115.11 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.9.0
    path: ~/.nvm/versions/node/v20.9.0/bin/node
  Yarn:
    version: 1.22.19
    path: ~/.nvm/versions/node/v20.9.0/bin/yarn
  npm:
    version: 10.1.0
    path: ~/.nvm/versions/node/v20.9.0/bin/npm
  Watchman:
    version: 2024.01.22.00
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.0
      - iOS 17.0
      - macOS 14.0
      - tvOS 17.0
      - watchOS 10.0
  Android SDK:
    API Levels:
      - "33"
      - "34"
    Build Tools:
      - 30.0.3
      - 33.0.0
      - 33.0.1
      - 34.0.0
      - 35.0.0
    System Images:
      - android-33 | Google APIs ARM 64 v8a
      - android-33 | Google APIs Intel x86_64 Atom
      - android-34 | Google APIs ARM 64 v8a
      - android-34 | Google APIs Intel x86_64 Atom
    Android NDK: Not Found
IDEs:
  Android Studio: 2022.3 AI-223.8836.35.2231.10811636
  Xcode:
    version: 15.0/15A240d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.5
    wanted: ^0.74.5
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs

[!] An error occurred while processing the post-install hook of the Podfile.

no implicit conversion of String into Array

/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:702:in `+'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:702:in `add_flag_to_map_with_inheritance'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:688:in `block (2 levels) in add_ndebug_flag_to_pods_in_release'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:683:in `each'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:683:in `block in add_ndebug_flag_to_pods_in_release'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:682:in `each'
/my-rn-project/node_modules/react-native/scripts/cocoapods/utils.rb:682:in `add_ndebug_flag_to_pods_in_release'
/my-rn-project/node_modules/react-native/scripts/react_native_pods.rb:304:in `react_native_post_install'
/my-rn-project/ios/Podfile:75:in `block (3 levels) in from_ruby'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-core-1.15.2/lib/cocoapods-core/podfile.rb:196:in `post_install!'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:1013:in `run_podfile_post_install_hook'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:1001:in `block in run_podfile_post_install_hooks'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/user_interface.rb:149:in `message'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:1000:in `run_podfile_post_install_hooks'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:337:in `block (2 levels) in create_and_save_projects'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer/xcode/pods_project_generator/pods_project_writer.rb:61:in `write!'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:336:in `block in create_and_save_projects'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/user_interface.rb:64:in `section'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:315:in `create_and_save_projects'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:307:in `generate_pods_project'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:183:in `integrate'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/installer.rb:170:in `install!'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/command/install.rb:52:in `run'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/claide-1.1.0/lib/claide/command.rb:334:in `run'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/lib/cocoapods/command.rb:52:in `run'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/gems/cocoapods-1.15.2/bin/pod:55:in `<top (required)>'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/bin/pod:25:in `load'
/opt/homebrew/Cellar/cocoapods/1.15.2/libexec/bin/pod:25:in `<main>'

Reproducer

No reproducer as it's not easy to reproduce, especially on recent project

Screenshots and Videos

No response

The text was updated successfully, but these errors were encountered:

react-native-bot · 2024-09-16T08:30:23Z

⚠️	Missing Reproducible Example
ℹ️	We could not detect a reproducible example in your issue report. Please provide either: If your bug is UI related: a Snack If your bug is build/update related: use our Reproducer Template. A reproducer needs to be in a GitHub repository under your username.

react-native-bot · 2024-09-16T08:30:34Z

⚠️	Missing Reproducible Example
ℹ️	We could not detect a reproducible example in your issue report. Please provide either: If your bug is UI related: a Snack If your bug is build/update related: use our Reproducer Template

cipolleschi · 2024-09-16T09:09:18Z

@MasGaNo thanks for the issue. The build setting can definitely be either String or Array. I'm not sure if that's due to different versions of Xcode, version of cocoapods, or the method used to actually read the project.

Anyway, I'd like to ask you a couple of things:

Could you try run bundle install and then install pods with bundle exec pod install? We use Ruby Bundler to ensure that all the cocoapods dependencies are aligned properly and that might fix your issue
Could you create a PR with the fix for this? If we handle them more solidly (using array methods when we receive an array, and strings methods when we receive a string) that would be better for everyone!

Thank you so much.

MasGaNo · 2024-09-17T12:14:54Z

@cipolleschi Thank you for the quick feedback.
Regarding the first point, I have some issue to run the bundle install command:

make: *** No rule to make target `/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.0.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/universal-darwin22/ruby/config.h', needed by `nkf.o'.  Stop.

I'm suspecting the fact I'm using the default Ruby 2.6 from MacOS/XCode and it may lead to some issues. I'm currently trying to replace it by a proper ruby and will let you know.

Regarding the fix itself, I just created the PR with the code snippet I put above and updated the test to support both Array and String value.

Thank you!

Summary: Following the discussion on #46505, this PR aims to allow mixte type configuration (String and/or Array of String) during the post installation of pods. ## Changelog:  [IOS] [FIXED] - allow pods mixte type settings on post-install Pull Request resolved: #46536 Test Plan: `packages/react-native/scripts/cocoapods/__tests__/utils-test.rb` test suits was updated to support array and works as expected Reviewed By: shwanton Differential Revision: D62870582 Pulled By: cipolleschi fbshipit-source-id: c0ace6d9d20e6609ceae5aafd236d97fc9e86ddf

MasGaNo · 2024-09-20T12:31:57Z

Could you try run bundle install and then install pods with bundle exec pod install? We use Ruby Bundler to ensure that all the cocoapods dependencies are aligned properly and that might fix your issue

Sorry for the late answer, but I finally managed to execute these commands, the first bundle install went smoothly, but the second one bundle exec pod install led to the same issue describe in my first post.

Hopefully, everything will be solved once the fix will be released 😄

cipolleschi · 2024-09-23T12:42:57Z

Thanks for testing that out.
At least we excluded that the problem was mismatch of Cocoapods and some ruby gem.
I guess that we can close the issue now that the commit that fixes has landed on main.
I also created a cherry-pick request for 0.76.

Summary: Following the discussion on #46505, this PR aims to allow mixte type configuration (String and/or Array of String) during the post installation of pods. ## Changelog:  [IOS] [FIXED] - allow pods mixte type settings on post-install Pull Request resolved: #46536 Test Plan: `packages/react-native/scripts/cocoapods/__tests__/utils-test.rb` test suits was updated to support array and works as expected Reviewed By: shwanton Differential Revision: D62870582 Pulled By: cipolleschi fbshipit-source-id: c0ace6d9d20e6609ceae5aafd236d97fc9e86ddf

MasGaNo added the Needs: Triage 🔍 label Sep 16, 2024

react-native-bot added Needs: Author Feedback Needs: Repro This issue could be improved with a clear list of steps to reproduce the issue. labels Sep 16, 2024

react-native-bot removed the Needs: Triage 🔍 label Sep 16, 2024

MasGaNo mentioned this issue Sep 17, 2024

fix(ios): allow pods mixte type settings on post-install #46536

Closed

github-actions bot added Needs: Attention Issues where the author has responded to feedback. and removed Needs: Author Feedback labels Sep 17, 2024

cipolleschi closed this as completed Sep 23, 2024

cortinico removed the Needs: Attention Issues where the author has responded to feedback. label Sep 23, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Error "no implicit conversion of String into Array" in Pod Install after migrating to RN > 0.74 (0.74.0 until 0.75.3) #46505

Error "no implicit conversion of String into Array" in Pod Install after migrating to RN > 0.74 (0.74.0 until 0.75.3) #46505

MasGaNo commented Sep 16, 2024 •

edited

Loading

react-native-bot commented Sep 16, 2024

react-native-bot commented Sep 16, 2024

cipolleschi commented Sep 16, 2024

MasGaNo commented Sep 17, 2024

MasGaNo commented Sep 20, 2024

cipolleschi commented Sep 23, 2024

Error "no implicit conversion of String into Array" in Pod Install after migrating to RN > 0.74 (0.74.0 until 0.75.3) #46505

Error "no implicit conversion of String into Array" in Pod Install after migrating to RN > 0.74 (0.74.0 until 0.75.3) #46505

Comments

MasGaNo commented Sep 16, 2024 • edited Loading

Description

Steps to reproduce

React Native Version

Affected Platforms

Output of npx react-native info

Stacktrace or Logs

Reproducer

Screenshots and Videos

react-native-bot commented Sep 16, 2024

react-native-bot commented Sep 16, 2024

cipolleschi commented Sep 16, 2024

MasGaNo commented Sep 17, 2024

MasGaNo commented Sep 20, 2024

cipolleschi commented Sep 23, 2024

MasGaNo commented Sep 16, 2024 •

edited

Loading

Output of `npx react-native info`