Skip to content

stty: support sane settings #8280

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

stty: support sane settings #8280

wants to merge 1 commit into from

Conversation

yuankunzhang
Copy link
Contributor

Closes #8228 .

@yuankunzhang yuankunzhang force-pushed the support-sane-settings branch 3 times, most recently from e9a00c3 to 0c97c7d Compare June 28, 2025 04:30
@yuankunzhang yuankunzhang force-pushed the support-sane-settings branch from 0c97c7d to a797e92 Compare June 28, 2025 04:37
@github-actions GitHub Actions
Copy link

GNU testsuite comparison:

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

@sylvestre
Copy link
Contributor

@willshuttleworth wdyt about that ? :)

@willshuttleworth
Copy link
Contributor

the parts that close #8228 seem good. the issue is that the pr includes changes beyond the scope of #8228, namely implementing the sane combination. #8256 is waiting to be merged and also implements the sane combination, but also all other combination settings that exist. i think if this gets reworked after #8256 is merged, that would be best.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

stty does not print non-default control character mappings
3 participants
@yuankunzhang @sylvestre @willshuttleworth