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 gradle compliation #925

Closed
wants to merge 1 commit into from

Conversation

amir-shalem
Copy link
Contributor

Summary:
Gradle is failing to compile yoga for two reasons:

  1. Ever since YogaNodeJNIPhantomRefs was introduced which uses DestructorThread.Destructor from fbjni which was the first direct Java dependency from yoga java to fbjni java code.

  2. Adding a missing gradle endpoint for testutil since it is now required for yoga unit-tests

Differential Revision: D17274226

Summary:
Pull Request resolved: facebook#925

Gradle is failing to compile yoga for two reasons:
1. Ever since `YogaNodeJNIPhantomRefs` was introduced which uses `DestructorThread.Destructor` from fbjni which was the first direct Java dependency from yoga java to fbjni java code.

2. Adding a missing gradle endpoint for `testutil` since it is now required for yoga unit-tests

Differential Revision: D17274226

fbshipit-source-id: 9a4919f1bb72ac1f52b6b32ce7acbff435d587e4
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 31de91b.

tobster-de pushed a commit to RECOMGmbH/Yoga that referenced this pull request Oct 30, 2020
Summary:
Pull Request resolved: facebook#925

Gradle is failing to compile yoga for two reasons:
1. Ever since `YogaNodeJNIPhantomRefs` was introduced which uses `DestructorThread.Destructor` from fbjni which was the first direct Java dependency from yoga java to fbjni java code.

2. Adding a missing gradle endpoint for `testutil` since it is now required for yoga unit-tests

Reviewed By: SidharthGuglani

Differential Revision: D17274226

fbshipit-source-id: 3df9648321162d34f81fd3675ca1474e8a1c6d3a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants