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

disk: Also pass stdin to install container #55

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

cgwalters
Copy link
Contributor

It's only by doing this that we end up calling into the code inside attach() that tries to ensure the terminal size matching. Otherwise we just get the default of 80 columns (and no dynamic SIGWINCH resizing).

This fixes the output in containers/bootc#655

It's only by doing this that we end up calling into the
code inside `attach()` that tries to ensure the terminal
size matching. Otherwise we just get the default of 80
columns (and no dynamic SIGWINCH resizing).

This fixes the output in containers/bootc#655

Signed-off-by: Colin Walters <[email protected]>
Copy link
Contributor

@jeckersb jeckersb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 confirmed this fixes the problem for me

@germag germag merged commit 70116ee into containers:main Jul 3, 2024
5 checks passed
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