duncs / clusterssh

Cluster SSH - Cluster Admin Via SSH
https://github.com/duncs/clusterssh/wiki
896 stars 79 forks source link

.clusterssh/config: terminal_size (esp. number of columns) is ignored #74

Open harridu opened 7 years ago

harridu commented 7 years ago

Apparently the terminal_size config parameter is ignored. I set it to 80x24 (the default), but "tput cols" reports only 77 colums or less, depending upon the font size. Can anybody reproduce?

duncs commented 7 years ago

It isn't ignored, but it probably doesn't work how you would expect.

The 80x24 is a guideline - the actual size is calculated from taking this terminal size, getting the font name and working out it's size, calculating the require terminal size by multiplying those together (as different sizes of fonts require different widths of terminals, and therefore different spacing between them), and then setting the terminal to the resulting figure. This figure is then also used for tiling calculations.

I will revisit this as and when I rewrite the window handling code.

xrat commented 1 year ago

I am currently using ClusterSSH 4.16 on Debian and I am also looking for a way to guarantee a terminal width of 80 columns. So, +1. Also, the manpage says: "the number of lines (24) will be decreased when resizing terminals for tiling, not the number of characters (80)."

xrat commented 1 year ago

It seems I figured out why my ClusterSSH xterms ended up with less than 80 columns: I am choosing a particular font and font-size by means of terminal_args, so I had removed the config setting terminal_font. Setting an appropriate value gets me 80 columns. The following 2 combinations (one for a slightly larger font) work for me:

terminal_args=-fa DejaVuSansMono -fs 9
terminal_font=7x13
terminal_args=-fa DejaVuSansMono -fs 10
terminal_font=8x13