[RN 0.49-rc.5] Fishhook error

[mobinni]

Is this a bug report?

No.

I tried to manually upgrade from 0.48.3 to 0.49-rc.5 due to the BLOB_URL error that happens in sed version.

jestjs/jest#4433
#15810

When upgrading everything went well until the following error occurred:

I tried adding:

pod "React", :path => react_native_path, :subspecs => ['DevSupport', 'fishhook']

However this did nothing for me, there is currently no documentation on how to integrate fishhook when manually upgrading. Can anyone help?

Have you read the Contributing Guidelines?

Yes

Environment

Environment:
OS: macOS Sierra 10.12.4
Node: 7.8.0
Yarn: 0.24.6
npm: 5.4.0
Xcode: Xcode 8.3.3 Build version 8E3004b

Packages: (installed)
react-native: 0.49-rc.5
react: 16.0.0-alpha.12

Target Platform: iOS (10.3)

Expected Behavior

Build succeeds

Actual Behavior

Build fails because it cannot find fishhook files

[mobinni]

update I figured it out to be a mis-import in reconnecting websocket, it links to

#import "<fishhook/fishhook.h>"

while it should link to

#import "fishhook.h"

[johannth]

This is a similar issue to #13198

I think this stems from Cocoapods not defining subspecs as modules/frameworks so there is no fishhook module and we can't do <fishhook/fishhook.h>. I've asked on the Cocoapods mailing list to confirm but I agree that the import is probably wrong and should either be:

#import <React/fishhook.h>

or

#import "fishhook.h"

An alternative in this case would be to depend on the fishhook Cocoapod instead of the embedded subspec. Then the import could stay as it is.

[ide]

I believe this is addressed in #16192 so I'm going to close this. That commit should go out with 0.49.1 in an hour or so.

[iwasrobbed-ks]

@johannth You can add a postinstall script to fix that issue into your package.json like

"scripts": {
        "postinstall": "sed -i '' 's#<fishhook/fishhook.h>#\"fishhook.h\"#g' ./node_modules/react-native/Libraries/WebSocket/RCTReconnectingWebSocket.m",
        // any other scripts here
},

[mobinni]

@rob-keepsafe It is fixed in 0.49.1, the issue doesn't occur on a lower version, so it should be good

[iwasrobbed-ks]

@mobinni @ide This still appears broken in 0.49.1 :\ The script above works fine for now

[mobinni]

@rob-keepsafe are you using pods, in which case you would need to pod update for it to take full effect.

I updated from 0.48 to 0.49.1 and it worked just fine after clearing my workspace (cmd shift alt k, cmd shift k), running a clean install, then pod updating

[iwasrobbed-ks]

I just installed React today for the first time, so it's been using 0.49.1 by default. The pods just point to the local podspecs in the node_modules directory as well

Deleting the node_modules/ and Pods/ directories, derived data, etc. doesn't work on my end

[macdoum1]

@mobinni Also installed React for the first time using cocoapods and followed the guide here: https://facebook.github.io/react-native/docs/integration-with-existing-apps.html Also seeing the missing fishhook import

[ide]

Reopening since people seem to be having issues still but am not working on it / don't know that anyone is, just to set expectations.

I'd also add that to remove potential discrepancies between dev environments, the patch that fixed this issue in some contexts was tested with Xcode 9 on Mac 10.13.

[macdoum1]

Anyone have any idea which older version does not have this issue? - Michael MacDougall

…

On Oct 9, 2017, at 5:47 PM, James Ide ***@***.***> wrote: Reopening since people seem to be having issues still but am not working on it / don't know that anyone is, just to set expectations. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[iwasrobbed-ks]

@macdoum1 No, but you can work around it for now via #16039 (comment)

[macdoum1]

Fairly new to react, but opened a PR here addressing the import: https://github.com/facebook/react-native/pull/16271/files