<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/139499>139499</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            [Support] Querying the terminal width on Linux is broken
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            llvm:support
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          Sirraide
      </td>
    </tr>
</table>

<pre>
    The Linux implementation of `Process::StandardOutColumns()`/`Process::StandardErrColumns()` (or, really, the underlying implementation of `getColumns()` in `Support/Unix/Process.inc`) seems wrong to me and doesn’t work on my system (Fedora 42) at least: both functions always return `0`. >From looking at the commit history around those functions, we used to query this via `ioctl()` with `TIOCGWINSZ`, but according to a3eb3d3d, this was ‘essentially disabled’ in https://reviews.llvm.org/D61326, and a3eb3d3d then removed the functionality entirely. Now, we rely exclusively on the ‘environment variable’ `COLUMNS` for this.

The problem with this is that `COLUMNS` is apparently not really an ‘environment variable’ but rather a shell variable, i.e. while accessible inside the shell (e.g. `echo $COLUMNS` works), it is not exported to any child processes spawned by that shell (at least not consistently on all systems). This is the case for instance on my system (Fedora 42) and I also tested it on a Debian 12 server; in both cases, running `std::getenv("COLUMNS")` always returns `nullptr`, for every combination of shell+terminal available to me.

As an aside, while a3eb3d3d also mentions that ‘Nobody has complained for one year’ wrt us relying on `COLUMNS` for this, there is exactly 1 combined use of these functions in the entire monorepo (and no tests for that matter but this also seems hard to test): the Clang driver calls `StandardErrColumns()` to try and determine a sensible error message length if `-fmessage-length` isn’t passed, which as a result is also very noticeably broken on my system. Interestingly, LLDB on Linux actually just uses `ioctl()` to figure out the terminal width and doesn’t bother calling either of these helpers.

I think we should undo these changes on some capacity and fall back on `ioctl()` if it is available and if `COLUMNS` is not defined. https://reviews.llvm.org/D61326 mentions that this functionality was disabled ‘for POSIX compatibility’, but I think we should still be able to make use of this on *some* systems (termios.h is part of the POSIX standard, and afaik `sys/ioctl.h` is always available on Linux at least). That said, I’m not an export on platform-dependent stuff like this so maybe there are issues with that, but the fact remains that these functions are just broken on some systems...

CC @MaskRay, @Endilll because iirc you’re familiar with this topic and @hubert-reinterpretcast because you reviewed D61326, but admittedly that was 6 years ago so I wouldn’t be surprised if you don’t remember much about this ;Þ

</pre>
<img width="1" height="1" alt="" src="http://email.email.llvm.org/o/eJyMV09z27oR_zTQZcccipRl66CDLEcdz-Qlr3XetNPbkliJeAIBFgtK5rfvLEhJdpLX5hSLAYHd378FkdkcHNFa3T-p--cZ9rHxYf1qQkCjaVZ5Pay_NQSfjevfwLSdpZZcxGi8A78Htcx_D74mZlVuVLl5jeg0Bv21j1tv-9axKh5VsVLLXBW7v1r9KYTvV4MqHn1QxRYCobWD_BUbgt5pCnYw7vDzag70w8FgnPzPa991PkRV7P5w5k0Vu6mUzLg6lbcCJmoZzsG7A0QPLQE6DdoTO_WpUI-5Wq0inH04gnfQDsADR2ql1h1pHxAWheyDESwhR1VuoPKxgX3vaimSAe0ZB4ZAsQ-prFwt8wx2wbdgvT9KYxhTq7VvWxOhMRx9GACD752G2Him24aCy5mgZ9JS8n96CgPExjCcDMr-xtfR3rA4m9jI428vX7d_--fLl9d_p963UPURsK590GbsHkuqSl3qEXnDcEaGCwyPxEwuGqEGtGGsLOkrRoJ4E2OXaC52qtgFOhk6c2btqc18OKhi97ycl8VSdheQL6dJ5w4Ctf5E6ce1VbQmDiCHBrJDBl_8eWpefgO91bZnc5K_vUtv3op1JxO8E63ACYORam_FqmW-_fr5j9--vApAex9Su5nKNyrfiPi74CtL7YhdgsIwxAbjd68aBuw6DOSiHcD5OGkX0P1qLcJCwNhQAARuyNrbomILJqMMzo2xJFwRs6ksgXFsNKWWx1dU8UjZIZPyqG48qGLxrkzRL4scZMMoVUup9CbuGFWEboC6MVZL63IMMXCHZ0caqmFs_XrSRexpl9o7NhxHBLwDtHYyiZyYwbcregQ1ipB9kPojupr-j6mchhdAyx4isVRqYjoCnqky6GBeAFM4UVDlkygwOU8OSSYJvXOibLXMOeoxfw4UyZ2SOYoLQEUxOeWDU1nec721XQyTYaRyOondat9Wxl1TKCGjiqdIoTUOLeAJjRUKx1CZlLVh0QUKdUnHI6sXG6Q-RSQpNUaxXST0xUsuQ4MsR3cWjRAj9XhHMBCGm6DOIULPySLSvXd_IfcpXwMJO_SGtRA4n1ojLQEjvcWG3qePwCxUjq6E1jsfqPNJFk6DG6ni6RSM0GKMFJLMk5FSm2PsNhiS-OQNoaDcpK23Ft0BdDAnClCjtYmK_zU5ZA_JS4luGkkgcRO50S4Ugg_QEjMeCCy5Q2zApOlxt58e342PR1e_D_8OmUlPhNUNIANCIO5tclLqJ4nC-WhqwsoOUAV_JPdB3Rm8uEiBOBp3GKfb58_PT7JmHLVYxz5lx589C4PEPwnz6GFvDn0g8P04M66aOxsdm5-OL7HFBKUogkz6eeW2IdtRuMTfi9DkjpKy3PjeahnAflpaN-gOxFI0-1YM3WEtIS3H7sX6FdbHSXPflW72U_bczCFvjTR8zFTJFU17kWH2q2PlO-skrX0cJDLNLoPrZi0R6u9fX1_-lZyF0VRGVl_hu0zKH3HhaKRhgqvR8Ug325gEkyo2gpQqNpdUFKsk0jxnjXTbYYgTG1MlPGn9Oir3aI4pxwZWxS4BmzWXATSm1g3Vm6Iul5IxhyXC0aRNX67ttQlsdNM0kJc7i3HvQ3unqSOnZWxx7Pd7sOZIY2OSVDhUNAUIphDhnvgyMTFecEsDHWuZjC2aGz0fU0V2SLq_OScJbMIsyyZ1bregFvlvyMd_YPKQWuSfnDY2MVGjoG9MqGHw_bXHIBW0xhoM7yZ69J2pE7xqkTd9RSHeBTJi0i5QrGXAXbYcfA-j8kjD7RqTblC6NTGSttOUFJUtUyQz4MELVC9wFsF8sCQB96ELRm5xZp8O0P79ikAttRUFaHsJncpfAlSVT2pbqtUnlW9mel3qVbnCGa3nD4uHvMzvH8pZs8ZV-bjS1Tyv82pFy9Xqsbh_qB4e7hclzRd6PzPrIi_u8_t5kS_ny3mZVVSWeb7Eal4tH6r9Si1yIcxenTZLDK_n5WqxWs0sVmQ5fUUUhSxR5YYv9-1CvizCWh7fVf2B1SK3huPNtbNook3fIJc7-v0z_F1usukq-mOsXTVteJLIrA92_TEbDiY2fZXVvlXFLtU0_nPXBf8n1fIhMKpUFbupjdO6-G8AAAD___4rYpo">