Skip to content

Commit

Permalink
Try fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Alex-PLACET committed Dec 11, 2024
1 parent 0678a54 commit 35f9189
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 20 deletions.
26 changes: 7 additions & 19 deletions .github/workflows/analysis.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,6 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v4

- name: Checkout nanoarrow
uses: actions/checkout@v4
with:
repository: apache/arrow-nanoarrow
path: dependencies/tests/nanoarrow

- name: Set conda environment
uses: mamba-org/setup-micromamba@main
with:
Expand All @@ -33,21 +27,15 @@ jobs:
init-shell: bash
cache-downloads: true

- name: Configure nanoarrow
working-directory: dependencies/tests/nanoarrow
- name: Configure using CMake
run: |
cmake -G Ninja \
-Bbuild \
${{matrix.sys.config-flags}} \
-DCMAKE_BUILD_TYPE:STRING=Release \
-DCMAKE_INSTALL_PREFIX=$CONDA_PREFIX
- name: Build nanoarrow
working-directory: dependencies/tests/nanoarrow/build
run: cmake --build . --config Release --target install

- name: Configure using CMake
run: cmake -G Ninja -Bbuild -DCMAKE_BUILD_TYPE:STRING=Release -DCMAKE_INSTALL_PREFIX=$CONDA_PREFIX -DBUILD_TESTS=ON -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
-Bbuild \
-DCMAKE_BUILD_TYPE:STRING=Release \
-DCMAKE_INSTALL_PREFIX=$CONDA_PREFIX \
-DBUILD_TESTS=ON \
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
-DFETCH_DEPENDENCIES_WITH_CMAKE=MISSING
- name: Run C++ analysis
uses: cpp-linter/cpp-linter-action@v2
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/osx.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ jobs:
-DBUILD_EXAMPLES=ON \
-DCMAKE_C_COMPILER_LAUNCHER=ccache \
-DCMAKE_CXX_COMPILER_LAUNCHER=ccache \
-Dnanoarrow_DIR=dependencies/tests/nanoarrow/build \
-DFETCH_DEPENDENCIES_WITH_CMAKE=MISSING
- name: Build
Expand Down

0 comments on commit 35f9189

Please sign in to comment.