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