amr/cl-tty
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use MY_TERM_COLS/LINES instead of COLUMNS/LINES

Browse Source 
SBCL unconditionally strips COLUMNS and LINES from the
environment. MY_TERM_COLS/MY_TERM_LINES bypass this filter.
This commit is contained in:
Amr Gharbeia
2026-05-14 14:55:37 -04:00
parent 8cb269dfee
commit 36fbe81441
2 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/backend/modern.lisp
View File

@@ -205,12 +205,11 @@ as a fallback when a keyword is not in *named-colors*.")
(rows (sb-alien:deref winsize 0)))
(values cols rows)))))
(sb-unix:unix-close tty-fd)))))
;; $COLUMNS/$LINES env vars — set by bash, may be available
;; even when ioctl fails (SBCL strips them in some contexts
;; but not all).
;; MY_TERM_COLS/MY_TERM_LINES — set by the passepartout script
;; before exec sbcl. SBCL strips COLUMNS/LINES but leaves these.
(ignore-errors
(let* ((cstr (sb-ext:posix-getenv "COLUMNS"))
(rstr (sb-ext:posix-getenv "LINES"))
(let* ((cstr (sb-ext:posix-getenv "MY_TERM_COLS"))
(rstr (sb-ext:posix-getenv "MY_TERM_LINES"))
(cols (when cstr (parse-integer cstr :junk-allowed t)))
(rows (when rstr (parse-integer rstr :junk-allowed t))))
(when (and cols rows (> cols 0) (> rows 0))

6
src/backend/simple.lisp
View File

@@ -63,10 +63,10 @@
(sb-alien:deref winsize 0))))
(sb-alien:free-alien winsize))
(sb-unix:unix-close tty-fd)))))
;; $COLUMNS/$LINES env vars
;; MY_TERM_COLS/MY_TERM_LINES — set by the passepartout script.
(ignore-errors
(let* ((cstr (sb-ext:posix-getenv "COLUMNS"))
(rstr (sb-ext:posix-getenv "LINES"))
(let* ((cstr (sb-ext:posix-getenv "MY_TERM_COLS"))
(rstr (sb-ext:posix-getenv "MY_TERM_LINES"))
(cols (when cstr (parse-integer cstr :junk-allowed t)))
(rows (when rstr (parse-integer rstr :junk-allowed t))))
(when (and cols rows (> cols 0) (> rows 0))