Skip to content

WIP stdbuf: add test_setvbuf_resolution #7972

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Ecordonnier
Copy link
Contributor

@Ecordonnier Ecordonnier commented May 22, 2025

This test verifies that stdbuf does work when cross-compiled. At the moment the test passes when compiled natively, but fails when compiled with cross-rs, due to #6591

This passes:

cargo test --features stdbuf test_stdbuf::test_setvbuf_resolution -- --nocapture

This fails:

cross test --target aarch64-unknown-linux-gnu --features stdbuf test_stdbuf::test_setvbuf_resolution -- --nocapture

@Ecordonnier Ecordonnier marked this pull request as draft May 22, 2025 20:29
@Ecordonnier Ecordonnier force-pushed the eco/stdbuf-regression-test branch from dc09788 to 22ae62b Compare May 22, 2025 20:57
This test verifies that stdbuf does work when cross-compiled. At the moment the test passes when compiled normally, but fails when compiled with cross-rs, due to uutils#6591

This passes:
```
cargo test --features stdbuf test_stdbuf::test_setvbuf_resolution -- --nocapture
```

This fails:
```
cross test --target aarch64-unknown-linux-gnu --features stdbuf test_stdbuf::test_setvbuf_resolution -- --nocapture
```

Signed-off-by: Etienne Cordonnier <ecordonnier@snap.com>
@Ecordonnier Ecordonnier force-pushed the eco/stdbuf-regression-test branch from 22ae62b to b73f9be Compare May 22, 2025 21:16
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant