fix: make sure we can build hermes when working against stable

facebook · Jul 19, 2022 · c120db2 · c120db2
1 parent a19578d
commit c120db2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/scripts/hermes/hermes-utils.js b/scripts/hermes/hermes-utils.js
@@ -192,10 +192,19 @@ function isOnAReleaseTag() {
 
 function shouldBuildHermesFromSource() {
   const hermesTag = readHermesTag();
+
+  const onReleaseBranch = isOnAReleaseBranch();
+  const onReleaseTag = isOnAReleaseTag();
+  const hermesTagIsDefault = hermesTag === DEFAULT_HERMES_TAG;
+
+  // Logging for CI Debugging
+  console.log(`[Hermes] On release branch? ${onReleaseBranch ? "Yes" : "No"}`)
+  console.log(`[Hermes] On release tag? ${onReleaseTag ? "Yes" : "No"}`)
+  console.log(`[Hermes] Hermes Tag is Default? ${hermesTagIsDefault ? "Yes" : "No"}`)
   return (
-    isOnAReleaseBranch() ||
-    isOnAReleaseTag() ||
-    hermesTag === DEFAULT_HERMES_TAG
+    onReleaseBranch ||
+    onReleaseTag ||
+    hermesTagIsDefault
   );
 }
 

diff --git a/scripts/hermes/prepare-hermes-for-build.js b/scripts/hermes/prepare-hermes-for-build.js
@@ -25,6 +25,7 @@ const {
 
 async function main() {
   if (!shouldBuildHermesFromSource()) {
+    console.log('[Hermes] Not building from Source')
     copyPodSpec();
     return;
   }