Skip to content

Commit 9b66166

Browse files
committed
ci.yml: Set RUSTDOCFLAGS to statically link doctests
Prior to Rust 1.89, these tests were just skipped when cross-compiling. Now, they are actually compiled and ran. Unfortunately, doctests don't use the normal RUSTFLAGS environment variable, so we also need to set RUSTDOCFLAGS or else the resulting dynamically linked executable will fail to run on a non-Android host. Upstream change: rust-lang/cargo#15462 Signed-off-by: Andrew Gunnerson <[email protected]>
1 parent f061879 commit 9b66166

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,8 @@ jobs:
5050
CARGO_TERM_COLOR: always
5151
# https://github.com/rust-lang/rust/issues/78210
5252
RUSTFLAGS: -C strip=symbols -C target-feature=+crt-static
53+
# https://github.com/rust-lang/cargo/pull/15462
54+
RUSTDOCFLAGS: -C target-feature=+crt-static
5355
TARGETS: ${{ join(matrix.artifact.targets, ' ') || matrix.artifact.name }}
5456
ANDROID_API: ${{ matrix.artifact.android_api }}
5557
strategy:

0 commit comments

Comments
 (0)