This is an automated email from the ASF dual-hosted git repository.

yuchanns pushed a commit to branch fix-build
in repository https://gitbox.apache.org/repos/asf/opendal-go-services.git

commit 7f89e1a45ed7acf8caded8ef00c44eda93f8e4be
Author: Hanchin Hsieh <m...@yuchanns.xyz>
AuthorDate: Thu Jul 17 15:28:11 2025 +0800

    fix: update build step to use OPENDAL_FEATURES for feature selection in 
cargo build, and remove unnecessary setup script for service features in 
build_artifacts workflow
    
    Signed-off-by: Hanchin Hsieh <m...@yuchanns.xyz>
---
 .github/workflows/build_artifacts.yaml | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/build_artifacts.yaml 
b/.github/workflows/build_artifacts.yaml
index 43c2f65..176e37b 100644
--- a/.github/workflows/build_artifacts.yaml
+++ b/.github/workflows/build_artifacts.yaml
@@ -75,20 +75,15 @@ jobs:
       - uses: actions/setup-python@v5
         with:
           python-version: "3.10"
-      - name: Setup Service
-        env:
-          OPENDAL_FEATURES: "layers-blocking,services-${{ matrix.service }}"
-        run: |
-          python -m pip install toml
-          python tools/.github/scripts/setup_features.py
       - name: Build ${{ matrix.service }} ${{ matrix.build.target }}
         working-directory: bindings/c
         env:
           SERVICE: ${{ matrix.service }}
           TARGET: ${{ matrix.build.target }}
+          OPENDAL_FEATURES: "opendal/services-${{ matrix.service }}"
           CC: ${{ matrix.build.cc }}
         run: |
-          cargo build --target $TARGET  --release
+          cargo build --target $TARGET --features $OPENDAL_FEATURES --release
           sudo apt install zstd
           zstd -22 ./target/$TARGET/release/libopendal_c.so -o 
./libopendal_c.$TARGET.so.zst
       - uses: actions/upload-artifact@v4

Reply via email to