Unix Console SetWindowSize implementation

[tmds]

In #75824 support was added for Console.SetWindowSize on Unix by using ioctl TIOCSWINSZ.

As I mentioned in #95654 (comment), I believe this is for informing the kernel about the actual size, and not for requesting a size.

This matches the behavior mentioned by @adamsitnik in this comment and the referenced stack overflow answer.

#75824 (comment) seems to sort of work because the kernel is told the terminal has a size of (80, 30). nvim gets that size when it asks the terminal size, and it then renders in that part of the terminal window only.

However, the window that shows the terminal doesn't show the terminal size changed.

I don't think this matches the Windows semantics, where (I assume) the window would resize or show scrollbars.

Based on this, I think we should not use TIOCSWINSZ to implement Console.SetWindowSize on Unix.

cc @adamsitnik @danmoseley @stephentoub @dotnet/area-system-console

Tagging subscribers to this area: @dotnet/area-system-console
See info in area-owners.md if you want to be subscribed.

Issue Details

---

In #75824 support was added for Console.SetWindowSize on Unix by using ioctl TIOCSWINSZ.

As I mentioned in #95654 (comment), I believe this is for informing the kernel about the actual size, and not for requesting a size.

This matches the behavior mentioned by @adamsitnik in this comment and the referenced stack overflow answer.

#75824 (comment) seems to sort of work because the kernel is told the terminal has a size of (80, 30). nvim gets that size when it asks the terminal size, and it then renders in that part of the terminal window only.

However, the window that shows the terminal doesn't show the terminal size changed.

I don't think this matches the Windows semantics, where (I assume) the window would resize or show scrollbars.

Based on this, I think we should not use TIOCSWINSZ to implement Console.SetWindowSize on Unix.

cc @adamsitnik @danmoseley @stephentoub @dotnet/area-system-console

Author:	tmds
Assignees:	-
Labels:	area-System.Console, untriaged
Milestone:	-

[tmds]

Based on the previous comment, I think we should revert #75824.
This means that the APIs will go back to throwing PNSE.

@adamsitnik @danmoseley @stephentoub @dotnet/area-system-console wdyt? are there any objections/concerns?

[jeffhandley]

No objections from me, @tmds. Thanks for digging into this. Feel free to submit a PR that reverts #75824, and we'll mark it as a breaking change.