Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix podspec to point to correct directory #139

Merged
merged 2 commits into from
Jan 10, 2018

Conversation

mgriepentrog
Copy link

If you try referencing this project's podspec, you get the following error:

[!] Failed to load 'ReactNativeSearchBar' podspec:
[!] Invalid `ReactNativeSearchBar.podspec` file: No such file or directory @ rb_sysopen - node_modules/react-native-search-bar/ios/package.json.

 #  from node_modules/react-native-search-bar/ios/ReactNativeSearchBar.podspec:2
 #  -------------------------------------------
 #  require "json"
 >  package = JSON.parse(File.read(File.join(__dir__, "package.json")))
 #
 #  -------------------------------------------

It's trying to load a package.json file in the iOS directory, which doesn't exist. Most open-source RN libraries have their podspec in the root folder [1][2], so I propose to move the podspec and update the source_files path accordingly.

If you try referencing this project's podspec, you get the following error:
```
[!] Failed to load 'ReactNativeSearchBar' podspec:
[!] Invalid `ReactNativeSearchBar.podspec` file: No such file or directory @ rb_sysopen - node_modules/react-native-search-bar/ios/package.json.

 #  from node_modules/react-native-search-bar/ios/ReactNativeSearchBar.podspec:2
 #  -------------------------------------------
 #  require "json"
 >  package = JSON.parse(File.read(File.join(__dir__, "package.json")))
 #
 #  -------------------------------------------
```

It's trying to load a `package.json` file in the iOS directory, which doesn't exist. Most open-source RN libraries have their podspec in the root folder.
Updating the `source_files` path to reflect the podspec move.
@iRoachie
Copy link
Collaborator

Thanks! Forgot this when I moved everything into the iOS folder.

@iRoachie iRoachie changed the base branch from master to next January 10, 2018 19:26
@iRoachie iRoachie changed the title Fix podspec Fix podspec to point to correct directory Jan 10, 2018
@iRoachie iRoachie merged commit ff593d3 into umhan35:next Jan 10, 2018
iRoachie added a commit that referenced this pull request Jan 10, 2018
Fix podspec to point to correct directory
@iRoachie
Copy link
Collaborator

Live in v3.3.1. Thanks again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants