Skip to content

stty: cleanup a hack for stty 51 panic - #15562

Merged
sylvestre merged 1 commit into
uutils:mainfrom
oech3:stty51panic
Oct 21, 2026
Merged

sylvestre merged 1 commit into
uutils:mainfrom
oech3:stty51panic

Conversation

@oech3

@oech3 oech3 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

@xtqqczze

Copy link
Copy Markdown
Collaborator

Should gate for Redox OS too

Right, stty now compiles successfully for Redox targets:

$ RUSTC_BOOTSTRAP=1 cargo check -q -Zbuild-std -p uu_stty --all-targets --target x86_64-unknown-redox

when it previously failed on main (de6c627):

error[E0609]: no field `line_discipline` on type `&nix::sys::termios::Termios`
   --> src/uu/stty/src/stty.rs:612:28
    |
612 |         let line = termios.line_discipline;
    |                            ^^^^^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `input_flags`, `output_flags`, `control_flags`, `local_flags`, `control_chars`

Comment thread src/uu/stty/src/stty.rs Outdated
Comment thread src/uu/stty/src/stty.rs Outdated
@oech3
oech3 requested a review from sylvestre October 21, 2026 14:31
@sylvestre
sylvestre merged commit 2fb0d76 into uutils:main Oct 21, 2026
98 checks passed
@sylvestre

Copy link
Copy Markdown
Contributor

Thanks for your PR

@oech3
oech3 deleted the stty51panic branch October 21, 2026 15:51
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.

3 participants