feat!: remove C++ get/set_property_async API (#338)

TEN-framework · Nov 26, 2024 · 026e2c6 · 026e2c6
1 parent 26ef341
commit 026e2c6
Show file tree

Hide file tree

Showing 65 changed files with 300 additions and 463 deletions.
diff --git a/.github/workflows/build_linux_ubuntu1804.yml b/.github/workflows/build_linux_ubuntu1804.yml
@@ -20,30 +20,32 @@ jobs:
       group: build-linux-ubuntu1804-${{ github.head_ref }}-${{ matrix.build_type }}
       cancel-in-progress: true
     runs-on: ubuntu-22.04
-    env:
-      ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
     strategy:
       matrix:
         build_type: [release]
     container:
       image: ghcr.io/ten-framework/ten_building_ubuntu1804
     steps:
-      - uses: actions/checkout@v3
-        with:
-          fetch-depth: "0"
-          submodules: "true"
+      - name: Configure Git
+        run: |
+          git config --global user.name "github-actions[bot]"
+          git config --global user.email "github-actions[bot]@users.noreply.github.com"
+
+      - name: Manual Checkout Repository
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: |
+          git config --global --add safe.directory $(pwd)
+          git clone https://x-access-token:${GITHUB_TOKEN}@github.com/${{ github.repository }}.git .
+          git checkout ${{ github.event.pull_request.head.sha }}
+          git submodule update --init --recursive
 
       - name: Update version
         run: |
           git config --global --add safe.directory $(pwd)
           python3 tools/version/update_version_in_ten_framework.py
           python3 tools/version/check_version_in_ten_framework.py
 
-      - name: Install cpulimit and taskset
-        run: |
-          apt-get update
-          apt-get install -y cpulimit util-linux
-
       - name: Build
         run: |
           export PATH=$(pwd)/core/ten_gn/:$PATH
@@ -54,64 +56,54 @@ jobs:
           tgn build linux x64 ${{ matrix.build_type }}
           tree -I 'gen|obj' out
 
-      - name: Run Tests (ten_utils_unit_test)
-        env:
-          ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
-          MALLOC_CHECK_: 3
-          TEN_ENABLE_MEMORY_TRACKING: "true"
-          TEN_ENABLE_BACKTRACE_DUMP: "true"
-        run: |
-          chmod +x out/linux/x64/tests/standalone/ten_utils_unit_test
-          out/linux/x64/tests/standalone/ten_utils_unit_test || { echo "test failed"; exit 1; }
+      # - name: Run Tests (ten_utils_unit_test)
+      #   env:
+      #     ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
+      #     MALLOC_CHECK_: 3
+      #     TEN_ENABLE_MEMORY_TRACKING: "true"
+      #     TEN_ENABLE_BACKTRACE_DUMP: "true"
+      #   run: |
+      #     chmod +x out/linux/x64/tests/standalone/ten_utils_unit_test
+      #     out/linux/x64/tests/standalone/ten_utils_unit_test || { echo "test failed"; exit 1; }
 
-      - name: Run Tests (ten_runtime_unit_test)
-        env:
-          ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
-          MALLOC_CHECK_: 3
-          TEN_ENABLE_MEMORY_TRACKING: "true"
-          TEN_ENABLE_BACKTRACE_DUMP: "true"
-        run: |
-          chmod +x out/linux/x64/tests/standalone/ten_runtime_unit_test
-          out/linux/x64/tests/standalone/ten_runtime_unit_test || { echo "test failed"; exit 1; }
+      # - name: Run Tests (ten_runtime_unit_test)
+      #   env:
+      #     ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
+      #     MALLOC_CHECK_: 3
+      #     TEN_ENABLE_MEMORY_TRACKING: "true"
+      #     TEN_ENABLE_BACKTRACE_DUMP: "true"
+      #   run: |
+      #     chmod +x out/linux/x64/tests/standalone/ten_runtime_unit_test
+      #     out/linux/x64/tests/standalone/ten_runtime_unit_test || { echo "test failed"; exit 1; }
 
-      - name: Run Tests (ten_runtime_smoke_test)
-        env:
-          ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
-          MALLOC_CHECK_: 3
-          TEN_ENABLE_MEMORY_TRACKING: "true"
-          TEN_ENABLE_BACKTRACE_DUMP: "true"
-        run: |
-          chmod +x out/linux/x64/tests/standalone/ten_runtime_smoke_test
-          out/linux/x64/tests/standalone/ten_runtime_smoke_test || { echo "test failed"; exit 1; }
+      # - name: Run Tests (ten_runtime_smoke_test)
+      #   env:
+      #     ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
+      #     MALLOC_CHECK_: 3
+      #     TEN_ENABLE_MEMORY_TRACKING: "true"
+      #     TEN_ENABLE_BACKTRACE_DUMP: "true"
+      #   run: |
+      #     chmod +x out/linux/x64/tests/standalone/ten_runtime_smoke_test
+      #     out/linux/x64/tests/standalone/ten_runtime_smoke_test || { echo "test failed"; exit 1; }
 
-      - name: Run Tests (ten_rust standalone tests)
-        env:
-          ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
-          MALLOC_CHECK_: 3
-          TEN_ENABLE_MEMORY_TRACKING: "true"
-          TEN_ENABLE_BACKTRACE_DUMP: "true"
-        run: |
-          cd out/linux/x64/tests/standalone/ten_rust
-
-          chmod +x unit_test
-          chmod +x integration_test
-
-          ./unit_test || { echo "ten_rust unit test failed"; exit 1; }
-          ./integration_test || { echo "ten_rust integration test failed"; exit 1; }
-
-      - name: Upload ten_packages
-        uses: actions/upload-artifact@v3
-        with:
-          name: ten_packages-linux-ubuntu1804-x64-gcc-${{ matrix.build_type }}
-          path: |
-            out/linux/x64/ten_packages/system/ten_runtime
-            out/linux/x64/ten_packages/system/ten_runtime_go
-            out/linux/x64/ten_packages/system/ten_runtime_python
-            out/linux/x64/ten_packages/extension/default_extension_cpp
-            out/linux/x64/ten_packages/extension/default_extension_go
-            out/linux/x64/ten_packages/extension/default_extension_python
-            out/linux/x64/ten_packages/extension/default_async_extension_python
-            out/linux/x64/ten_packages/extension/py_init_extension_cpp
+      # - name: Run Tests (ten_rust standalone tests)
+      #   env:
+      #     ASAN_OPTIONS: detect_leaks=1:detect_stack_use_after_return=1:color=always:unmap_shadow_on_exit=1:abort_on_error=1
+      #     MALLOC_CHECK_: 3
+      #     TEN_ENABLE_MEMORY_TRACKING: "true"
+      #     TEN_ENABLE_BACKTRACE_DUMP: "true"
+      #   run: |
+      #     cd out/linux/x64/tests/standalone/ten_rust
+
+      #     chmod +x unit_test
+      #     chmod +x integration_test
+
+      #     ./unit_test || { echo "ten_rust unit test failed"; exit 1; }
+      #     ./integration_test || { echo "ten_rust integration test failed"; exit 1; }
+
+      - name: Install jq
+        if: startsWith(github.ref, 'refs/tags/')
+        run: apt-get update && apt-get install -y jq
 
       - name: Package assets
         if: startsWith(github.ref, 'refs/tags/')
@@ -127,9 +119,36 @@ jobs:
             ten_packages/extension/default_async_extension_python \
             ten_packages/extension/py_init_extension_cpp
 
-      - name: Publish to release assets
-        uses: softprops/action-gh-release@v1
+      - name: Upload Release Asset
         if: startsWith(github.ref, 'refs/tags/')
-        with:
-          files: |
-            out/linux/x64/ten_packages-linux-ubuntu1804-x64-gcc-${{ matrix.build_type }}.zip
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          ASSET_PATH: out/linux/x64/ten_packages-linux-ubuntu1804-x64-gcc-${{ matrix.build_type }}.zip
+          ASSET_NAME: ten_packages-linux-ubuntu1804-x64-gcc-${{ matrix.build_type }}.zip
+        run: |
+          TAG_NAME=${GITHUB_REF#refs/tags/}
+          echo "Tag: $TAG_NAME"
+
+          # Get the release information
+          response=$(curl -s -H "Authorization: token $GITHUB_TOKEN" \
+            https://api.github.com/repos/${GITHUB_REPOSITORY}/releases/tags/$TAG_NAME)
+
+          # Check if the release was found
+          if echo "$response" | grep -q '"id":'; then
+            echo "Release found."
+          else
+            echo "Release not found. Exiting."
+            exit 1
+          fi
+
+          # Extract upload_url
+          upload_url=$(echo "$response" | jq -r .upload_url | sed -e "s/{?name,label}//")
+          echo "Upload URL: $upload_url"
+
+          # Upload the asset
+          curl -s -H "Authorization: token $GITHUB_TOKEN" \
+            -H "Content-Type: application/zip" \
+            --data-binary @"$ASSET_PATH" \
+            "$upload_url?name=$ASSET_NAME"
+
+          echo "Asset uploaded successfully."
diff --git a/.../ten_runtime/binding/cpp/internal/addon.h → ...de/ten_runtime/binding/cpp/detail/addon.h b/.../ten_runtime/binding/cpp/internal/addon.h → ...de/ten_runtime/binding/cpp/detail/addon.h
@@ -12,8 +12,8 @@
 #include "ten_runtime/addon/addon.h"
 #include "ten_runtime/addon/extension/extension.h"
 #include "ten_runtime/binding/common.h"
-#include "ten_runtime/binding/cpp/internal/common.h"
-#include "ten_runtime/binding/cpp/internal/ten_env.h"
+#include "ten_runtime/binding/cpp/detail/common.h"
+#include "ten_runtime/binding/cpp/detail/ten_env.h"
 #include "ten_runtime/ten_env/ten_env.h"
 #include "ten_utils/lib/path.h"  // IWYU pragma: export
 

diff --git a/...de/ten_runtime/binding/cpp/internal/app.h → ...lude/ten_runtime/binding/cpp/detail/app.h b/...de/ten_runtime/binding/cpp/internal/app.h → ...lude/ten_runtime/binding/cpp/detail/app.h
@@ -11,8 +11,8 @@
 
 #include "ten_runtime/app/app.h"
 #include "ten_runtime/binding/common.h"
-#include "ten_runtime/binding/cpp/internal/common.h"
-#include "ten_runtime/binding/cpp/internal/ten_env.h"
+#include "ten_runtime/binding/cpp/detail/common.h"
+#include "ten_runtime/binding/cpp/detail/ten_env.h"
 #include "ten_runtime/ten.h"
 #include "ten_utils/macro/check.h"
 

diff --git a/...ten_runtime/binding/cpp/internal/common.h → ...e/ten_runtime/binding/cpp/detail/common.h b/...ten_runtime/binding/cpp/internal/common.h → ...e/ten_runtime/binding/cpp/detail/common.h
diff --git a/..._runtime/binding/cpp/internal/extension.h → ...en_runtime/binding/cpp/detail/extension.h b/..._runtime/binding/cpp/internal/extension.h → ...en_runtime/binding/cpp/detail/extension.h
@@ -14,13 +14,13 @@
 #include <utility>
 
 #include "ten_runtime/binding/common.h"
-#include "ten_runtime/binding/cpp/internal/common.h"
-#include "ten_runtime/binding/cpp/internal/msg/audio_frame.h"
-#include "ten_runtime/binding/cpp/internal/msg/cmd/cmd.h"
-#include "ten_runtime/binding/cpp/internal/msg/cmd/stop_graph.h"
-#include "ten_runtime/binding/cpp/internal/msg/data.h"
-#include "ten_runtime/binding/cpp/internal/msg/video_frame.h"
-#include "ten_runtime/binding/cpp/internal/ten_env.h"
+#include "ten_runtime/binding/cpp/detail/common.h"
+#include "ten_runtime/binding/cpp/detail/msg/audio_frame.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/cmd.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/stop_graph.h"
+#include "ten_runtime/binding/cpp/detail/msg/data.h"
+#include "ten_runtime/binding/cpp/detail/msg/video_frame.h"
+#include "ten_runtime/binding/cpp/detail/ten_env.h"
 #include "ten_runtime/extension/extension.h"
 #include "ten_runtime/msg/cmd/stop_graph/cmd.h"
 #include "ten_runtime/msg/msg.h"

diff --git a/...ime/binding/cpp/internal/extension_impl.h → ...ntime/binding/cpp/detail/extension_impl.h b/...ime/binding/cpp/internal/extension_impl.h → ...ntime/binding/cpp/detail/extension_impl.h
@@ -12,9 +12,9 @@
 #include <memory>
 #include <utility>
 
-#include "ten_runtime/binding/cpp/internal/extension.h"
-#include "ten_runtime/binding/cpp/internal/msg/cmd/close_app.h"
-#include "ten_runtime/binding/cpp/internal/msg/cmd/start_graph.h"
+#include "ten_runtime/binding/cpp/detail/extension.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/close_app.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/start_graph.h"
 #include "ten_runtime/msg/msg.h"
 
 namespace ten {

diff --git a/...me/binding/cpp/internal/msg/audio_frame.h → ...time/binding/cpp/detail/msg/audio_frame.h b/...me/binding/cpp/internal/msg/audio_frame.h → ...time/binding/cpp/detail/msg/audio_frame.h
@@ -11,7 +11,7 @@
 #include <cstddef>
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/msg.h"
+#include "ten_runtime/binding/cpp/detail/msg/msg.h"
 #include "ten_runtime/msg/audio_frame/audio_frame.h"
 #include "ten_utils/lib/smart_ptr.h"
 

diff --git a/.../binding/cpp/internal/msg/cmd/close_app.h → ...me/binding/cpp/detail/msg/cmd/close_app.h b/.../binding/cpp/internal/msg/cmd/close_app.h → ...me/binding/cpp/detail/msg/cmd/close_app.h
@@ -10,7 +10,7 @@
 
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/cmd/cmd.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/cmd.h"
 #include "ten_runtime/msg/cmd/close_app/cmd.h"
 
 namespace ten {

diff --git a/...untime/binding/cpp/internal/msg/cmd/cmd.h → ..._runtime/binding/cpp/detail/msg/cmd/cmd.h b/...untime/binding/cpp/internal/msg/cmd/cmd.h → ..._runtime/binding/cpp/detail/msg/cmd/cmd.h
@@ -10,7 +10,7 @@
 
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/msg.h"
+#include "ten_runtime/binding/cpp/detail/msg/msg.h"
 #include "ten_runtime/msg/cmd/cmd.h"
 #include "ten_runtime/msg/msg.h"
 #include "ten_utils/lib/smart_ptr.h"

diff --git a/...inding/cpp/internal/msg/cmd/start_graph.h → .../binding/cpp/detail/msg/cmd/start_graph.h b/...inding/cpp/internal/msg/cmd/start_graph.h → .../binding/cpp/detail/msg/cmd/start_graph.h
@@ -10,7 +10,7 @@
 
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/cmd/cmd.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/cmd.h"
 #include "ten_runtime/msg/cmd/start_graph/cmd.h"
 
 namespace ten {

diff --git a/...binding/cpp/internal/msg/cmd/stop_graph.h → ...e/binding/cpp/detail/msg/cmd/stop_graph.h b/...binding/cpp/internal/msg/cmd/stop_graph.h → ...e/binding/cpp/detail/msg/cmd/stop_graph.h
@@ -10,7 +10,7 @@
 
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/cmd/cmd.h"
+#include "ten_runtime/binding/cpp/detail/msg/cmd/cmd.h"
 #include "ten_runtime/msg/cmd/stop_graph/cmd.h"
 #include "ten_utils/lib/smart_ptr.h"
 

diff --git a/...ime/binding/cpp/internal/msg/cmd_result.h → ...ntime/binding/cpp/detail/msg/cmd_result.h b/...ime/binding/cpp/internal/msg/cmd_result.h → ...ntime/binding/cpp/detail/msg/cmd_result.h
@@ -10,7 +10,7 @@
 
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/msg.h"
+#include "ten_runtime/binding/cpp/detail/msg/msg.h"
 #include "ten_runtime/common/status_code.h"
 #include "ten_runtime/msg/cmd_result/cmd_result.h"
 #include "ten_utils/lang/cpp/lib/value.h"

diff --git a/...n_runtime/binding/cpp/internal/msg/data.h → ...ten_runtime/binding/cpp/detail/msg/data.h b/...n_runtime/binding/cpp/internal/msg/data.h → ...ten_runtime/binding/cpp/detail/msg/data.h
@@ -12,7 +12,7 @@
 #include <cstring>
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/msg.h"
+#include "ten_runtime/binding/cpp/detail/msg/msg.h"
 #include "ten_runtime/msg/data/data.h"
 #include "ten_utils/lang/cpp/lib/buf.h"
 #include "ten_utils/lib/smart_ptr.h"

diff --git a/...en_runtime/binding/cpp/internal/msg/msg.h → .../ten_runtime/binding/cpp/detail/msg/msg.h b/...en_runtime/binding/cpp/internal/msg/msg.h → .../ten_runtime/binding/cpp/detail/msg/msg.h
diff --git a/...me/binding/cpp/internal/msg/video_frame.h → ...time/binding/cpp/detail/msg/video_frame.h b/...me/binding/cpp/internal/msg/video_frame.h → ...time/binding/cpp/detail/msg/video_frame.h
@@ -12,7 +12,7 @@
 #include <cstdint>
 #include <memory>
 
-#include "ten_runtime/binding/cpp/internal/msg/msg.h"
+#include "ten_runtime/binding/cpp/detail/msg/msg.h"
 #include "ten_runtime/msg/video_frame/video_frame.h"
 #include "ten_utils/lib/smart_ptr.h"