-
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
[RNTester] Fix BUCK build #27826
[RNTester] Fix BUCK build #27826
Conversation
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.
@passy has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was exported from Phabricator. Differential Revision: D19496769 |
Summary: This fixes a build failure with buck introduced with facebook#27729. The internal and external buck overlays diverged in how fbjni was imported. The Buck re-export here ensures that the targets resolve both internall and externally. ## Changelog [Android] [Fixed] - RNTester Buck Build Pull Request resolved: facebook#27826 Test Plan: buck fetch rntester buck build rntester Reviewed By: cpojer Differential Revision: D19496769 Pulled By: passy fbshipit-source-id: 9537c72b9d6fef14639eb7d79caec017cef2eb17
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.
@passy has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was successfully merged by @passy in a3cb377. When will my fix make it into a release? | Upcoming Releases |
Summary: This fixes a build failure with buck introduced with #27729. The internal and external buck overlays diverged in how fbjni was imported. The Buck re-export here ensures that the targets resolve both internall and externally. ## Changelog [Android] [Fixed] - RNTester Buck Build Pull Request resolved: #27826 Test Plan: buck fetch rntester buck build rntester Reviewed By: jknoxville Differential Revision: D19496769 Pulled By: passy fbshipit-source-id: d699a5f64f691ed375cfc7a9d6a5a6f6e36ba283
Summary: This fixes a build failure with buck introduced with facebook#27729. The internal and external buck overlays diverged in how fbjni was imported. The Buck re-export here ensures that the targets resolve both internall and externally. ## Changelog [Android] [Fixed] - RNTester Buck Build Pull Request resolved: facebook#27826 Test Plan: buck fetch rntester buck build rntester Reviewed By: jknoxville Differential Revision: D19496769 Pulled By: passy fbshipit-source-id: d699a5f64f691ed375cfc7a9d6a5a6f6e36ba283
Summary
This fixes a build failure with buck introduced with #27729. The internal and external buck overlays diverged in how fbjni was imported. The Buck re-export here ensures that the targets resolve both internall and externally.
Changelog
[Android] [Fixed] - RNTester Buck Build
Test Plan
buck fetch rntester
buck build rntester
Waiting for Circle CI build to confirm.