From 1db96a3c469b872e851553207e5420d54afc731a Mon Sep 17 00:00:00 2001 From: Janic Duplessis Date: Tue, 30 Jul 2019 01:17:10 -0700 Subject: [PATCH] Add missing hermes include (#25872) Summary: The Hermes react executor c++ code is not currently getting built. The include was probably forgotten when merging hermes on master. The 0.60 branch does have it https://github.com/facebook/react-native/blob/0.60-stable/ReactAndroid/src/main/jni/react/jni/Android.mk#L72 ## Changelog [Android] [Fixed] - Add missing hermes include Pull Request resolved: https://github.com/facebook/react-native/pull/25872 Test Plan: Run an app with hermes enabled on RN master Differential Revision: D16559354 Pulled By: cpojer fbshipit-source-id: 95cfbf65481c2f0d0e5c53ca35f327753e7b99ae --- ReactAndroid/src/main/jni/react/jni/Android.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/ReactAndroid/src/main/jni/react/jni/Android.mk b/ReactAndroid/src/main/jni/react/jni/Android.mk index e687f40c170847..7446f296d752a3 100644 --- a/ReactAndroid/src/main/jni/react/jni/Android.mk +++ b/ReactAndroid/src/main/jni/react/jni/Android.mk @@ -77,3 +77,4 @@ include $(REACT_SRC_DIR)/turbomodule/core/jni/Android.mk # $(call import-module,jscexecutor) include $(REACT_SRC_DIR)/jscexecutor/Android.mk +include $(REACT_SRC_DIR)/../hermes/reactexecutor/Android.mk