Skip to content

Commit 8860bbb

Browse files
committed
update
1 parent 6ed4983 commit 8860bbb

File tree

5 files changed

+24
-13
lines changed

5 files changed

+24
-13
lines changed

.github/actions/setup-c++/action.yml

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
name: "Setup C++"
2+
description: "Toolchain setup and Initial compilation"
3+
4+
runs:
5+
using: "composite"
6+
steps:
7+
- name: Setup C++
8+
uses: aminya/setup-cpp@v1
9+
with:
10+
compiler: llvm
11+
sccache: true
12+
cmake: true
13+
ninja: true
14+
vcpkg: true

.github/actions/setup-rust/action.yml

+1-10
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,4 @@ runs:
3939
echo "CARGO_INCREMENTAL=0" >> $GITHUB_ENV
4040
- name: Export Path
4141
shell: bash
42-
run: echo "PATH=$PATH" >> $GITHUB_ENV
43-
44-
- name: Setup Cpp
45-
uses: aminya/setup-cpp@v1
46-
with:
47-
compiler: llvm
48-
vcvarsall: true
49-
cmake: true
50-
ninja: true
51-
vcpkg: true
42+
run: echo "PATH=$PATH" >> $GITHUB_ENV

.github/workflows/ci.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,7 @@ jobs:
148148
- uses: ./.github/actions/setup-rust
149149
with:
150150
targets: ${{matrix.config.target || ''}}
151+
- uses: ./.github/actions/setup-c++
151152
- name: Install cargo-hack
152153
uses: taiki-e/install-action@cargo-hack
153154
- name: Rust Build (${{matrix.config.name}})
@@ -161,6 +162,7 @@ jobs:
161162
- uses: ./.github/actions/cleanup
162163
- uses: rui314/setup-mold@v1
163164
- uses: ./.github/actions/setup-rust
165+
- uses: ./.github/actions/setup-c++
164166
- uses: taiki-e/install-action@cargo-hack
165167
- uses: taiki-e/install-action@cargo-minimal-versions
166168
- run: cargo minimal-versions check --direct --workspace --ignore-private
@@ -173,6 +175,7 @@ jobs:
173175
- uses: ./.github/actions/cleanup
174176
- uses: rui314/setup-mold@v1
175177
- uses: ./.github/actions/setup-rust
178+
- uses: ./.github/actions/setup-c++
176179
- name: Rust Lint - Format
177180
run: cargo fmt --all --check
178181
- name: Rustc check
@@ -207,6 +210,7 @@ jobs:
207210
- uses: ./.github/actions/cleanup
208211
- uses: rui314/setup-mold@v1
209212
- uses: ./.github/actions/setup-rust
213+
- uses: ./.github/actions/setup-c++
210214
- name: Install grcov
211215
if: github.ref == 'refs/heads/develop'
212216
run: cargo install grcov
@@ -312,7 +316,7 @@ jobs:
312316
with:
313317
tool: nextest
314318
- name: Run all tests with Miri
315-
run: cargo miri nextest run --no-fail-fast --workspace --exclude vortex-file --exclude vortex-layout --exclude vortex-sampling-compressor --exclude vortex-fsst --exclude vortex-array --exclude vortex-dtype --exclude vortex-expr --exclude vortex-scalar
319+
run: cargo miri nextest run --no-fail-fast --workspace --exclude vortex-file --exclude vortex-layout --exclude vortex-sampling-compressor --exclude vortex-fsst --exclude vortex-array --exclude vortex-dtype --exclude vortex-expr --exclude vortex-scalar --exclude vortex-duckdb
316320
# For now, we only run Miri against known "fiddly" crates.
317321
if: false
318322
- name: Run Miri

vortex-duckdb/Cargo.toml

+3-1
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,14 @@ path = "src/lib.rs"
1919
bench = false
2020

2121
[dependencies]
22-
duckdb = { workspace = true, features = ["vtab-full", "bundled"] }
2322
itertools = { workspace = true }
2423
vortex-array = { workspace = true }
2524
vortex-dtype = { workspace = true }
2625
vortex-error = { workspace = true }
2726

27+
[target.'cfg(not(target_arch = "wasm32"))'.dependencies]
28+
duckdb = { workspace = true, features = ["vtab-full", "bundled"] }
29+
2830
[dev-dependencies]
2931

3032
[lints]

vortex-duckdb/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
extern crate core;
1+
#![cfg(target_arch = "wasm32")]
22

33
mod convert;
44

0 commit comments

Comments
 (0)