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

":CFBundleIdentifier", Does Not Exist - Mitigating confusion around it #21303

Closed
hramos opened this issue Sep 24, 2018 · 2 comments
Closed

":CFBundleIdentifier", Does Not Exist - Mitigating confusion around it #21303

hramos opened this issue Sep 24, 2018 · 2 comments
Labels
DX Issues concerning how the developer experience can be improved. 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled p: Facebook Partner: Facebook Partner Platform: iOS iOS applications. Resolution: Locked This issue was locked by the bot. Tech: Bundler 📦 This issue is related to the bundler (Metro, Haul, etc) used. Type: Discussion Long running discussion.

Comments

@hramos
Copy link
Contributor

hramos commented Sep 24, 2018

Print: Entry, ":CFBundleIdentifier", Does Not Exist pops up constantly in GitHub issues. This is the last line of output for a number of unrelated issues.

This message will appear in the logs whenever a React Native project fails to compile or link.

The Print: Entry, ":CFBundleIdentifier", Does Not Exist error, by itself, is too generic to give direction, and does not indicate whether the error is in React Native, third party libraries, or application code.

With this in mind, we are automatically marking issues that mention the :CFBundleIdentifier error as duplicates of this issue. This issue will remain locked in order to avoid confusing matters any further, while still allowing core contributors to discuss ways to mitigate the problem.

Troubleshooting

If you're landing on this issue because you've observed the :CFBundleIdentifier message in your output, please read the following carefully:

  • This error typically indicates a failure to compile or link the app.
  • If you're having problems, look at the full build output, not just the last few lines.
  • Open the project in Xcode and hit ⌘B to build it. The build errors in Xcode might help find the actual cause of the failure.

Troubleshooting by Example

For an actual example of how such an issue can be investigated, take a look at
https://gist.github.com/mhorowitz/ea67d8e1793e803b2ca273395ac12ed2. The important lines are 9-10:

../../../ios/Pods/leveldb-library/port/win/stdint.h:11:2: error: This file should only be included when compiling with MSVC.
#error This file should only be included when compiling with MSVC.
 ^
/tmp/meeat-app/node_modules/react-native-google-signin/ios/RNGoogleSignin/RNGoogleSignin.m:68:45: warning: 'sendAppEventWithName:body:' is deprecated: Subclass RCTEventEmitter instead [-Wdeprecated-declarations]
        return [self.bridge.eventDispatcher sendAppEventWithName:@"RNGoogleSignInError"

Those messages appear about 80 lines before the :CFBundleIdentifier line. As you can see, the hint for the root cause can sometimes be found much earlier in the logs. The root cause in the above example is completely unrelated to a missing :CFBundleIdentifier entry.

Always provide a full repro

As you can see in the example above, having a full repro is very helpful. Someone trying to debug can see all the relevant errors and input files. Even with the errors in the example above, without being able to see the full list of dependences and look at project files, someone could not have identified the actual bug.

Related issues

For Discussion

Let's come up with a plan to mitigate confusion here. If this error constantly shows up for any number of unrelated failures, we can likely do a better job of surfacing the actual issue rather than simply leading people on a wild goose chase.

@hramos hramos added Platform: iOS iOS applications. Type: Discussion Long running discussion. Core Team DX Issues concerning how the developer experience can be improved. 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled labels Sep 24, 2018
@facebook facebook locked as resolved and limited conversation to collaborators Sep 24, 2018
@react-native-bot react-native-bot added the Tech: Bundler 📦 This issue is related to the bundler (Metro, Haul, etc) used. label Sep 24, 2018
@kelset kelset pinned this issue Dec 14, 2018
@kelset kelset changed the title Mitigating confusion around _Print: Entry, ":CFBundleIdentifier", Does Not Exist_ ":CFBundleIdentifier", Does Not Exist - Mitigating confusion around it Dec 14, 2018
@grabbou
Copy link
Contributor

grabbou commented Jan 21, 2019

I am opening an issue in CLI to track this react-native-community/cli#107.

There's definitely a lot of work to be done inside "run-ios" command, but I think it's a priority from the DX perspective.

@hramos hramos unpinned this issue Jan 23, 2019
@hramos hramos added Partner p: Facebook Partner: Facebook and removed Core Team labels Mar 8, 2019
@cpojer
Copy link
Contributor

cpojer commented Mar 19, 2019

This should be fixed with 0.59 as it was fixed in the latest RN cli: react-native-community/cli@6526bdd

Yay!

@cpojer cpojer closed this as completed Mar 19, 2019
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Mar 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DX Issues concerning how the developer experience can be improved. 📮Known Issues This indicates an issue that refers to a bug or limitation of RN that is not currently being handled p: Facebook Partner: Facebook Partner Platform: iOS iOS applications. Resolution: Locked This issue was locked by the bot. Tech: Bundler 📦 This issue is related to the bundler (Metro, Haul, etc) used. Type: Discussion Long running discussion.
Projects
None yet
Development

No branches or pull requests

4 participants