Skip to content

build: search terminal info libraries#579

Merged
rewolff merged 1 commit into
traviscross:masterfrom
Komzpa:darafei/search-terminal-info-lib
May 25, 2026
Merged

build: search terminal info libraries#579
rewolff merged 1 commit into
traviscross:masterfrom
Komzpa:darafei/search-terminal-info-lib

Conversation

@Komzpa

@Komzpa Komzpa commented May 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Search terminal-info libraries when curses support is enabled.

Some distributions split terminal state functions out of ncurses/ncursesw, so linking curses code can require an additional tinfow, tinfo, or termcap library. This is a refreshed version of the old #368 idea, with tinfow checked first so the current ncursesw path keeps the matching wide-character terminal-info library when needed.

Supersedes #368.

Authorship note: the branch keeps Jeroen Roovers as the commit author for the refreshed #368 libtinfo lookup, with the current-tree adaptation co-authored by Darafei.

Validation

  • ./bootstrap.sh
  • ./configure --without-gtk --without-jansson
  • verified configure checks for wprintw, initscr, and raw
  • make -j $(nproc)
  • git diff --check
  • uv run --python 3.9 --with flake8==3.9.2 --with flake8-bandit==2.1.2 --with bandit==1.7.2 --with pbr==7.0.3 python -m flake8 .

The build still prints the existing ui/split.c snprintf truncation warnings on this compiler; those warnings are present on the current code path and are not introduced by this PR.

Refresh the libtinfo lookup originally proposed in PR traviscross#368 for the current ncursesw/ncurses configure path.

Co-authored-by: Darafei Praliaskouski <me@komzpa.net>
@Komzpa Komzpa force-pushed the darafei/search-terminal-info-lib branch from 4163460 to 20dd4bc Compare May 8, 2026 03:40
@rewolff rewolff merged commit 146fce7 into traviscross:master May 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants