Skip to content

Commit 4c741be

Browse files
maxholmanclaude
andcommitted
chore: remove CARGO_TARGET_DIR override, add build.rs duplication comment
CARGO_TARGET_DIR=target/musl was only needed for cloud cache key isolation. Cross already outputs to target/<triple>/. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 6f98ec3 commit 4c741be

3 files changed

Lines changed: 7 additions & 5 deletions

File tree

.github/workflows/pr.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ jobs:
5353
if: runner.environment != 'self-hosted'
5454
with:
5555
shared-key: musl
56-
workspaces: . -> target/musl
5756

5857
- name: Install cross
5958
run: |
@@ -70,7 +69,7 @@ jobs:
7069
run: docker build -t wallhack-cross:x86_64-unknown-linux-musl -f Dockerfile.cross-musl .
7170

7271
- name: Test (musl — primary)
73-
run: CARGO_TARGET_DIR=target/musl cross test --release --target x86_64-unknown-linux-musl
72+
run: cross test --release --target x86_64-unknown-linux-musl
7473

7574
build-musl:
7675
needs: [test-musl]
@@ -85,7 +84,6 @@ jobs:
8584
if: runner.environment != 'self-hosted'
8685
with:
8786
shared-key: musl
88-
workspaces: . -> target/musl
8987

9088
- name: Install cross
9189
run: |
@@ -102,14 +100,14 @@ jobs:
102100
run: docker build -t wallhack-cross:x86_64-unknown-linux-musl -f Dockerfile.cross-musl .
103101

104102
- name: Build slim (musl — primary)
105-
run: CARGO_TARGET_DIR=target/musl cross build --release --target x86_64-unknown-linux-musl -p wallhack-cli --no-default-features --features slim
103+
run: cross build --release --target x86_64-unknown-linux-musl -p wallhack-cli --no-default-features --features slim
106104

107105
- name: Check slim binary size (musl — primary)
108106
run: ./bench/check_bloat.sh --no-build --only=slim-musl
109107
continue-on-error: true
110108

111109
- name: Build default (musl — primary)
112-
run: CARGO_TARGET_DIR=target/musl cross build --release --target x86_64-unknown-linux-musl -p wallhack-cli
110+
run: cross build --release --target x86_64-unknown-linux-musl -p wallhack-cli
113111

114112
- name: Check default binary size (musl — primary)
115113
run: ./bench/check_bloat.sh --no-build --only=default-musl

crates/cli/build.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// NOTE: This WallhackBuildEnv impl is duplicated in crates/mcp/build.rs.
2+
// build.rs files cannot share code via library crates.
13
use std::collections::BTreeMap;
24

35
use vergen_gitcl::{

crates/mcp/build.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// NOTE: This WallhackBuildEnv impl is duplicated in crates/cli/build.rs.
2+
// build.rs files cannot share code via library crates.
13
use std::collections::BTreeMap;
24

35
use vergen_gitcl::{

0 commit comments

Comments
 (0)