follow react-native-node-api standard approach

Built with:

```sh
./build_all_react_native.sh
```

Author: shirakaba

NativeScript/CMakeLists.txt

@@ -1,7 +1,6 @@
 cmake_minimum_required(VERSION 3.15)
 
 # Metadata
-
 project(NativeScript CXX OBJCXX)
 
 set(NAME NativeScript)
@@ -17,7 +16,6 @@ set(COMMON_FLAGS "-O3 -Wno-shorten-64-to-32")
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${COMMON_FLAGS}")
 
 # Arguments
-
 set(TARGET_PLATFORM "macos" CACHE STRING "Target platform for the Objective-C bridge")
 set(TARGET_ENGINE "v8" CACHE STRING "Target JS engine for the NativeScript runtime")
 set(METADATA_SIZE 0 CACHE STRING "Size of embedded metadata in bytes")
@@ -56,6 +54,10 @@ elseif(TARGET_PLATFORM STREQUAL "macos")
   set(TARGET_PLATFORM_SPEC "macos-arm64_x86_64")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DTARGET_PLATFORM_MACOS")
 
+  # Unlike "ios", "macos" doesn't seem to produce a NativeScript.framework.dSYM
+  # by default. We can produce it by setting these variables.
+  set(CMAKE_XCODE_ATTRIBUTE_DEBUG_INFORMATION_FORMAT "dwarf-with-dsym")
+  set(CMAKE_XCODE_ATTRIBUTE_DWARF_DSYM_FOLDER_PATH "${CMAKE_CURRENT_BINARY_DIR}/$<CONFIG>")
 else()
   message(FATAL_ERROR "Unknown target platform: ${TARGET_PLATFORM}")
   return()
@@ -87,18 +89,16 @@ endif()
 if(ENABLE_JS_RUNTIME)
   set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -DENABLE_JS_RUNTIME")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DENABLE_JS_RUNTIME")
-elseif(TARGET_PLATFORM_MACOS)
-  # If building a generic library for macOS, we'll build a dylib instead of a framework
-  unset(BUILD_FRAMEWORK)
+else()
   set(GENERIC_NAPI TRUE)
 endif()
 
 message(STATUS "TARGET_PLATFORM = ${TARGET_PLATFORM}")
 message(STATUS "TARGET_ENGINE = ${TARGET_ENGINE}")
 message(STATUS "ENABLE_JS_RUNTIME = ${ENABLE_JS_RUNTIME}")
+message(STATUS "GENERIC_NAPI = ${GENERIC_NAPI}")
 
 # Set up sources
-
 include_directories(
   ./
   ../metadata-generator/include
@@ -201,12 +201,14 @@ if(ENABLE_JS_RUNTIME)
     )
 
     set(SOURCE_FILES ${SOURCE_FILES}
+
       # quickjs
       napi/quickjs/source/cutils.c
       napi/quickjs/source/libregexp.c
       napi/quickjs/source/libbf.c
       napi/quickjs/source/libunicode.c
       napi/quickjs/source/quickjs.c
+
       # napi
       napi/quickjs/quickjs-api.c
       napi/quickjs/jsr.cpp
@@ -223,7 +225,6 @@ if(ENABLE_JS_RUNTIME)
       napi/jsc/jsc-api.cpp
       napi/jsc/jsr.cpp
     )
-
   endif()
 else()
   include_directories(
@@ -239,7 +240,6 @@ if(BUILD_CLI_BINARY)
 endif()
 
 # Find SDK
-
 find_program(XCODEBUILD_EXECUTABLE xcodebuild)
 
 if(METADATA_SIZE EQUAL 0)
@@ -265,7 +265,6 @@ set(CMAKE_OSX_SYSROOT "${CMAKE_OSX_SYSROOT_INT}" CACHE INTERNAL "")
 message(STATUS "SDK = ${CMAKE_OSX_SYSROOT}")
 
 # Build targets
-
 if(BUILD_CLI_BINARY)
   add_executable(${NAME} ${SOURCE_FILES})
 else()
@@ -301,11 +300,22 @@ endif()
 
 if(TARGET_PLATFORM_MACOS)
   # add_custom_command(TARGET ${NAME} POST_BUILD
-  #   COMMAND /usr/libexec/PlistBuddy -c "Add :LSMinimumSystemVersion string ${CMAKE_OSX_DEPLOYMENT_TARGET}" $<TARGET_FILE_DIR:${NAME}>/Resources/Info.plist
+  # COMMAND /usr/libexec/PlistBuddy -c "Add :LSMinimumSystemVersion string ${CMAKE_OSX_DEPLOYMENT_TARGET}" $<TARGET_FILE_DIR:${NAME}>/Resources/Info.plist
   # )
 
+  # Convert the Info.plist from binary format to XML format.
+  # This seemed to unblock a build error when using in React Native.
+  add_custom_command(TARGET ${NAME} POST_BUILD
+    COMMAND plutil -convert xml1 $<TARGET_FILE_DIR:${NAME}>/Resources/Info.plist
+  )
   set(METADATA_FILE "metadata.macos.nsmd")
 elseif(TARGET_PLATFORM_IOS)
+  # Convert the Info.plist from binary format to XML format.
+  # This seemed to unblock a build error when using in React Native.
+  add_custom_command(TARGET ${NAME} POST_BUILD
+    COMMAND plutil -convert xml1 $<TARGET_FILE_DIR:${NAME}>/Info.plist
+  )
+
   if(TARGET_PLATFORM_SIM)
     set(METADATA_FILE "metadata.ios-sim.nsmd")
   else()
@@ -362,7 +372,7 @@ if(TARGET_ENGINE_JSC)
 endif()
 
 if(TARGET_ENGINE_V8)
-  if (TARGET_PLATFORM_MACOS)
+  if(TARGET_PLATFORM_MACOS)
     target_link_directories(
       ${NAME}
       PRIVATE
@@ -414,12 +424,12 @@ if(TARGET_ENGINE_QUICKJS)
   )
 endif()
 
-# if (GENERIC_NAPI)
-target_link_options(
-  ${NAME}
-  PRIVATE
-  "-Wl"
-  "-undefined"
-  "dynamic_lookup"
-)
-# endif()
+if(GENERIC_NAPI)
+  target_link_options(
+    ${NAME}
+    PRIVATE
+    "-Wl"
+    "-undefined"
+    "dynamic_lookup"
+  )
+endif()

build_all_ios.sh

@@ -1,5 +1,29 @@
 #!/bin/bash
 set -e
+source "$(dirname "$0")/build_utils.sh"
+
+TARGET_ENGINE=${TARGET_ENGINE:=some}
+BUILD_IPHONE=$(to_bool ${BUILD_IPHONE:=true})
+BUILD_SIMULATOR=$(to_bool ${BUILD_SIMULATOR:=true})
+BUILD_MACOS=$(to_bool ${BUILD_MACOS:=false})
+EMBED_METADATA=$(to_bool ${EMBED_METADATA:=false})
+
+# See build_nativescript.sh for all supported flags. This parent script is only
+# interested in intercepting --no-engine.
+for arg in $@; do
+  case $arg in
+    --v8|--quickjs|--jsc|--hermes) TARGET_ENGINE=some ;;
+    --sim|--simulator) BUILD_SIMULATOR=true ;;
+    --no-sim|--no-simulator) BUILD_SIMULATOR=false ;;
+    --iphone|--device) BUILD_IPHONE=true ;;
+    --no-iphone|--no-device) BUILD_IPHONE=false ;;
+    --macos) BUILD_MACOS=true ;;
+    --no-macos) BUILD_MACOS=false ;;
+    --no-engine) TARGET_ENGINE=none ;;
+    --embed-metadata) EMBED_METADATA=true ;;
+    *) ;;
+  esac
+done
 
 rm -rf ./dist
 # don't run if NO_UPDATE_VERSION is set
@@ -8,7 +32,37 @@ rm -rf ./dist
   # ./update_version.sh
 # fi
 ./build_metadata_generator.sh
-./build_nativescript.sh --no-vision $1 $2
-./build_tklivesync.sh --no-vision
-./prepare_dSYMs.sh
-./build_npm_ios.sh
+
+if $EMBED_METADATA; then
+  checkpoint "Generating metadata, as --embed-metadata was passed..."
+
+  if $BUILD_IPHONE; then
+    checkpoint "Generating metadata for iOS (physical device)..."
+    npm run metagen ios
+  fi
+  if $BUILD_SIMULATOR; then
+    checkpoint "Generating metadata for iOS (simulator)..."
+    npm run metagen ios-sim
+  fi
+  if $BUILD_MACOS; then
+    checkpoint "Generating metadata for macOS..."
+    npm run metagen macos
+  fi
+
+  checkpoint "... All metadata generated!"
+fi
+
+./build_nativescript.sh --no-vision $1 $2 $3 $4 $5 $6 $7 $8 $9
+
+if [[ "$TARGET_ENGINE" == "none" ]]; then
+  # If you're building *with* --no-engine, you're trying to make an npm release
+  # of a workspace under ./packages/*, like @nativescript/ios-node-api.
+  echo "Skipping build_npm_ios.sh due to --no-engine flag."
+  echo "build_all_ios.sh finished!"
+else
+  # If you're building *without* --no-engine, you're trying to make an npm
+  # release of the root-level workspace, @nativescript/ios.
+  ./build_tklivesync.sh --no-vision
+  ./prepare_dSYMs.sh
+  ./build_npm_ios.sh
+fi

build_all_react_native.sh

@@ -0,0 +1,3 @@
+#!/bin/bash
+
+./build_all_ios.sh --no-engine --embed-metadata --macos

build_all_vision.sh

@@ -1,10 +1,35 @@
 #!/bin/bash
 set -e
 
+TARGET_ENGINE=${TARGET_ENGINE:=v8} # default to v8 for compat
+
+# See build_nativescript.sh for all supported flags. This parent script is only
+# interested in --no-engine.
+for arg in $@; do
+  case $arg in
+    --v8) TARGET_ENGINE=v8 ;;
+    --quickjs) TARGET_ENGINE=quickjs ;;
+    --jsc) TARGET_ENGINE=jsc ;;
+    --hermes) TARGET_ENGINE=hermes ;;
+    --no-engine) TARGET_ENGINE=none ;;
+    *) ;;
+  esac
+done
+
 rm -rf ./dist
 ./update_version.sh
 ./build_metadata_generator.sh
-./build_nativescript.sh --no-catalyst --no-iphone --no-sim
+./build_nativescript.sh --no-catalyst --no-iphone --no-sim $1
 ./build_tklivesync.sh --no-catalyst --no-iphone --no-sim
 ./prepare_dSYMs.sh
-./build_npm_vision.sh
+
+if [[ "$TARGET_ENGINE" == "none" ]]; then
+  # If you're building *with* --no-engine, you're trying to make an npm release
+  # of a workspace under ./packages/*, like @nativescript/ios-node-api.
+  echo "Skipping build_npm_vision.sh due to --no-engine flag."
+  echo "build_all_vision.sh finished!"
+else
+  # If you're building *without* --no-engine, you're trying to make an npm
+  # release of the root-level workspace, @nativescript/ios.
+  ./build_npm_vision.sh
+fi