Skip to content

Comments

Enable pipe on reception of SIGTTIN#1436

Merged
squell merged 1 commit intomainfrom
signal-ttin-enable
Feb 9, 2026
Merged

Enable pipe on reception of SIGTTIN#1436
squell merged 1 commit intomainfrom
signal-ttin-enable

Conversation

@squell
Copy link
Member

@squell squell commented Feb 6, 2026

This is would finish the remaining item of #1263. This fixes the sudo cat /dev/tty | cat issue.

Maybe @mrmonopoly-cyber can look at this PR to see if this fixes the original #1263 issue as well.

@squell squell added this to the Community Support milestone Feb 6, 2026
@squell squell linked an issue Feb 6, 2026 that may be closed by this pull request
@mrmonopoly-cyber
Copy link
Contributor

I checked issue #1263 but this fix alone does not fix the issue and this is also true for sudo cat /dev/tty | cat. Both SingnalState and this patch are needed to fix the issue at the moment. Btw thank you for the help with this issue. Because this is exams period i'm having quite an hard time to work on personal projects.

@squell
Copy link
Member Author

squell commented Feb 6, 2026

Yes, this fix is intended to go on top of that; i.e. most of #1263 is fixed by #1420, this is a minor fixup. But thanks for checking whether this alone would suffice (I wouldn't have guessed it does)

@squell squell merged commit e171b3b into main Feb 9, 2026
18 checks passed
@squell squell deleted the signal-ttin-enable branch February 9, 2026 11:35
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.

Problems with piping into sudo-rs

3 participants