-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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 detection of android emulator #35111
Conversation
at some point, the `Build.FINGERPRINT` of the emulator has changed and no longer includes `generic` as an option. It apparently now, on all platforms, includes `google/sdk_gphone` at the beginning.
Base commit: 6a43faf |
Base commit: 76f7084 |
PR build artifact for 53fd80a is ready. |
Thanks for sending this @deecewan |
Hey @cortinico - I tested this using the debugger to ensure that the Here's a screenshot of the I ran this by patching the file in my |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cipolleschi can you import this one?
@GijsWeterings has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: at some point, the `Build.FINGERPRINT` of the emulator has changed and no longer includes `generic` as an option. It apparently now, on all platforms, includes `google/sdk_gphone` at the beginning. Older emulators may still include `generic` in the `Build.FINGERPRINT` so we should maintain that check too. ## Changelog <!-- Help reviewers and the release process by writing your own changelog entry. For an example, see: https://reactnative.dev/contributing/changelogs-in-pull-requests --> [Android] [Fixed] - Fix android emulator detection for packager host Pull Request resolved: facebook#35111 Test Plan: When running on a modern emulator, ensure that the packager attempts to load from `10.0.2.2`. Reviewed By: cipolleschi Differential Revision: D41266071 Pulled By: GijsWeterings fbshipit-source-id: 43115dbde6a411fe2ebde23e720dff4812a4309f
Summary
at some point, the
Build.FINGERPRINT
of the emulator has changed and no longer includesgeneric
as an option.It apparently now, on all platforms, includes
google/sdk_gphone
at the beginning. Older emulators may still includegeneric
in theBuild.FINGERPRINT
so we should maintain that check too.Changelog
[Android] [Fixed] - Fix android emulator detection for packager host
Test Plan
When running on a modern emulator, ensure that the packager attempts to load from
10.0.2.2
.