Skip to content

Commit d4838a1

Browse files
committed
ci: fix publish tier ordering and add missing build dependencies
Reorganize the crates.io publish tiers to correctly reflect dependency order, moving nodedb-sql and nodedb-columnar into tier 3 alongside the other tier-2 dependents. Add libcurl4-openssl-dev and libsasl2-dev to the build dependency list for both publish jobs. Update tier count comment from 5 to 4 to match the current tier structure.
1 parent df4f568 commit d4838a1

1 file changed

Lines changed: 11 additions & 9 deletions

File tree

.github/workflows/release.yml

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ jobs:
7373

7474
# ── Publish crates to crates.io ──────────────────────────────────────────────
7575
# Requires secret: CARGO_REGISTRY_TOKEN
76-
# All 5 tiers run in a single job with is_published checks and wait_for polling
76+
# All 4 tiers run in a single job with is_published checks and wait_for polling
7777
# so re-running a failed publish never double-publishes an already-indexed crate.
7878
publish-crates:
7979
name: Publish to crates.io
@@ -92,7 +92,8 @@ jobs:
9292
run: |
9393
sudo apt-get update
9494
sudo apt-get install -y --no-install-recommends \
95-
cmake clang libclang-dev pkg-config protobuf-compiler perl
95+
cmake clang libclang-dev pkg-config protobuf-compiler perl \
96+
libcurl4-openssl-dev libsasl2-dev
9697
9798
- name: Set version from tag
9899
run: |
@@ -105,7 +106,7 @@ jobs:
105106
# For pre-release versions, pin internal dep requirements so semver
106107
# "0.0.0" doesn't fail to match "0.0.0-beta.1"
107108
if [[ "$VERSION" == *-* ]]; then
108-
sed -i -E 's/(nodedb-(codec|bridge|wal|mem|crdt|raft|fts|types|vector|graph|spatial|strict|client|columnar|cluster|query) = \{ [^}]*version = )"[^"]*"/\1"='"$VERSION"'"/' Cargo.toml
109+
sed -i -E 's/(nodedb-(codec|bridge|wal|mem|crdt|raft|fts|types|vector|graph|spatial|strict|client|columnar|cluster|query|sql) = \{ [^}]*version = )"[^"]*"/\1"='"$VERSION"'"/' Cargo.toml
109110
echo "Updated internal dep versions to =$VERSION"
110111
fi
111112
@@ -120,11 +121,11 @@ jobs:
120121
run: |
121122
# Tier 1: no internal NodeDB dependencies
122123
TIER1="nodedb-codec nodedb-bridge nodedb-wal nodedb-mem nodedb-crdt nodedb-raft nodedb-fts"
123-
# Tier 2: depends on tier 1
124-
TIER2="nodedb-types nodedb-vector nodedb-graph nodedb-spatial nodedb-strict nodedb-client"
125-
# Tier 3: depends on tier 2
126-
TIER3="nodedb-columnar nodedb-cluster"
127-
# Tier 4: depends on tier 3
124+
# Tier 2: depends on tier 1 only
125+
TIER2="nodedb-types"
126+
# Tier 3: depends on tier 1-2
127+
TIER3="nodedb-vector nodedb-graph nodedb-spatial nodedb-strict nodedb-client nodedb-sql nodedb-columnar nodedb-cluster"
128+
# Tier 4: depends on tier 3 (nodedb-spatial)
128129
TIER4="nodedb-query"
129130
is_published() {
130131
curl -sf \
@@ -206,7 +207,8 @@ jobs:
206207
run: |
207208
sudo apt-get update
208209
sudo apt-get install -y --no-install-recommends \
209-
cmake clang libclang-dev pkg-config protobuf-compiler perl
210+
cmake clang libclang-dev pkg-config protobuf-compiler perl \
211+
libcurl4-openssl-dev libsasl2-dev
210212
211213
- name: Set version from tag
212214
run: |

0 commit comments

Comments
 (0)