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

Included GLog.podspec broken on case sensitive file system #16181

Closed
athei opened this issue Oct 3, 2017 · 12 comments
Closed

Included GLog.podspec broken on case sensitive file system #16181

athei opened this issue Oct 3, 2017 · 12 comments
Labels
Platform: iOS iOS applications. Resolution: Fixed A PR that fixes this issue has been merged. Resolution: Locked This issue was locked by the bot.

Comments

@athei
Copy link

athei commented Oct 3, 2017

Resolution

Fixed in 0.54, update your podfile:

pod 'glog', :podspec => 'node_modules/react-native/third-party-podspecs/glog.podspec'

Original Bug Report

Is this a bug report?

yes

Have you read the Contributing Guidelines?

yes

Environment

Environment:
OS: macOS High Sierra 10.13
Node: 8.6.0
Yarn: 1.1.0
npm: 5.3.0
Watchman: 4.9.0
Xcode: Xcode 9.0 Build version 9A235
Android Studio: 2.3 AI-162.4069837

Packages: (wanted => installed)
react: 16.0.0-alpha.12 => 16.0.0-alpha.12
react-native: ^0.48.2 => 0.48.3

Target Platform: iOS (10.3)

Steps to Reproduce

  1. Place your the source of your react-native based project on a case sensitive file system on macOS (HFS+ or APFS).
  2. Manage your dependencies using cocoapods instead of react-native link.

Here is a minimal Podfile:

platform :ios, '10.3'
inhibit_all_warnings!

target 'MyiOSProject' do
  # React dependencies
  pod 'Yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
  pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
  pod 'GLog', :podspec => '../node_modules/react-native/third-party-podspecs/GLog.podspec'
  pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
  
  # React and its submodules
  pod 'React', :path => '../node_modules/react-native', :subspecs => [
    'Core',
    'CxxBridge',
    'DevSupport',
    'RCTPushNotification',
    'RCTText',
    'RCTImage',
    'RCTAnimation',
    'RCTNetwork'
  ]
end
  1. pod install

  2. Try to build the resulting xcworkspace using Xcode.

Expected Behavior

When opening the xcworkspace and press build it should build without errors.

Actual Behavior

I get a build error saying .../Pods/Folly/folly/Range.h:33:10: 'glog/logging.h' file not found

screen shot 2017-10-03 at 12 56 07

How to fix

Just fix your supplied GLog.podspec to have spec.name = 'glog' instead of spec.name = 'GLog'. The name of the library is glog not GLog. Because of this misspelling cocoapods names the subdirectory in the Headers folder wrongly and therefore the inclusion error on case sensitive systems.

@ide
Copy link
Contributor

ide commented Nov 3, 2017

Anyone want to send a PR for this?

@maxkomarychev
Copy link
Contributor

maxkomarychev commented Nov 23, 2017

@athei could you please try my fork? you can install it using this line in your package.json:

"react-native": "git+ssh://git@github.com:maxkomarychev/react-native#glog-folly-case"

important keep in mind this fork is based on tag 0.50.3

and then modify your Podfile to:

pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'

@MuhammadAftabAslam
Copy link

Hi
I am facing the same issue. I have tried everything and wasted last 3 days
Please help me!!!!

@39otrebla
Copy link

Not fixed as of react-native@0.52.0 , very annoying

@CFKevinRef
Copy link
Contributor

CFKevinRef commented Jan 18, 2018

I addressed this here: https://github.com/facebook/react-native/pull/17476/files#diff-cf374a993404a1abd4a5d4972c6f87caR13

I think this is a lower-impact fix.

Would this be worth splitting out into its own PR?
Edit: #17476 has been merged, should fix this.

@anuragdadheech
Copy link

anuragdadheech commented Mar 7, 2018

This is fixed in react-native@0.54.0
Update your Podfile to:
pod 'glog', :podspec => 'node_modules/react-native/third-party-podspecs/glog.podspec'

@aksswami
Copy link

This has been fixed and docs are also updated, but updated docs are not reflected on the react-native docs website.

@hramos hramos added Platform: iOS iOS applications. Resolution: Fixed A PR that fixes this issue has been merged. labels Mar 29, 2018
@hramos
Copy link
Contributor

hramos commented Mar 29, 2018

Anyone want to send a PR to fix the docs? https://github.com/facebook/react-native-website

@hramos hramos closed this as completed Mar 29, 2018
@hramos
Copy link
Contributor

hramos commented Mar 29, 2018

Disregard. The docs are already up to date, @simonbengtsson updated master three weeks ago: http://facebook.github.io/react-native/docs/next/integration-with-existing-apps.html

I've updated the 0.54 version to include this as well.

@ahmetilhann
Copy link

This problem has been fixed at 0.54.4v. I noticed today.

@sumanlamsal
Copy link

If you are using React Native <0.54, you will get the following error:

"The name of the given podspec GLog doesn't match the expected one glog"

Use the following line instead:

#pod 'GLog', :podspec => "#{rn_path}/third-party-podspecs/GLog.podspec"

@yasinugrl
Copy link

yasinugrl commented Sep 5, 2018

I get this issue in RN: 0.55.4 and fixed it by:

pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'

@facebook facebook locked as resolved and limited conversation to collaborators Mar 29, 2019
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Mar 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Platform: iOS iOS applications. Resolution: Fixed A PR that fixes this issue has been merged. Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

No branches or pull requests