Skip to content
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

Screen buffer #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Screen buffer #58

wants to merge 2 commits into from

Conversation

legionus
Copy link

@legionus legionus commented Dec 2, 2021

When a client attaches to an existing session, the session command output is not available to him. The problem is not noticeable when the program (top, man, etc.) in the session is able to redraw its output.

But this does not work with non-interactive utilities. The output of commands in bash will not be available to the client connecting to the session.

To solve this, we add a buffer that contains the last few lines of the session output. When a new client attaches, the contents of this buffer are sent to him.

This PR could address #57, #32 and maybe more.

Fixes: a99e46f ("Simplify I/O handling")
Signed-off-by: Alexey Gladkov <[email protected]>
When a client attaches to an existing session, the session command
output is not available to him. The problem is not noticeable when the
program (top, man, etc.) in the session is able to redraw its output.

But this does not work with non-interactive utilities. The output of
commands in bash will not be available to the client connecting to the
session.

To solve this, we add a buffer that contains the last few lines of the
session output. When a new client attaches, the contents of this buffer
are sent to him.

Signed-off-by: Alexey Gladkov <[email protected]>
@megalithic
Copy link

this is brilliant!

any chances of merging @martanne?

@legionus
Copy link
Author

@megalithic I haven't seen activity here in years. I maintain my fork.

@megalithic
Copy link

@megalithic I haven't seen activity here in years. I maintain my fork.

thanks @legionus! i've switched to building from your repo from source. cheers!

@legionus
Copy link
Author

thanks @legionus! i've switched to building from your repo from source. cheers!

@megalithic If you have any other suggestions, I'm ready to add to my fork. I hope in the future I will have a chance to discuss them with upstream.

@littlewing7
Copy link

hi @martanne ,

many thanks for you work!
there is a reason this patch cannot be merged on abduco ?

cheers

Max

@Atry
Copy link

Atry commented Aug 27, 2024

@martanne What do you think?

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.

4 participants