rexpect icon indicating copy to clipboard operation
rexpect copied to clipboard

Set default terminal size to 80x25

Open tailhook opened this issue 5 years ago • 3 comments

It looks like otherwise the default is 0, which crashes some programs.

I would make it customizable, but I'm not sure how to make API for that.

Also see #10

tailhook avatar May 12 '20 15:05 tailhook

thanks for the PR @tailhook and sorry that I didn't respond so far. I'm giving this project some love again, and would like to also look into this PR and maybe a way to customize the terminal size.

I'd like to play with it, what's your use case? or asked differently: what program crashes currently without the terminal size?

philippkeller avatar May 24 '20 09:05 philippkeller

I'd like to play with it, what's your use case? or asked differently: what program crashes currently without the terminal size?

This is the crash: https://github.com/edgedb/edgedb-cli/runs/666845514?check_suite_focus=true Note, the crash is actually in rustyline library. And that may involve some PR which isn't yet landed into a master.

Now as I'm thinking about it more, maybe it's better to fix rustyline to treat that as "infinite width". But being able to specify terminals size in rexpect makes sense anyway.

tailhook avatar May 25 '20 10:05 tailhook

Also see this this thread on the similar issue: https://github.com/containers/libpod/issues/351

tailhook avatar May 26 '20 14:05 tailhook

Hi! I am closing this PR now as it seems to be no longer relevant to you. Feel free to open a new one if you're still running into issues!

matthiasbeyer avatar Sep 12 '22 11:09 matthiasbeyer