Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ncurses: include terminfo database in package #1452

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ianthomas23
Copy link
Collaborator

This modifies the ncurses recipe to include terminal database information, in other words runtime configuration files, in the share directory. This is needed for the upcoming vim recipe which needs to use the terminfo/x/xterm-256color to operate correctly. Rather than just building and shipping that one file, I'm including all of the database files as I expect more will be needed at some point in the future.

The changes to the recipe are at the configure stage, removing the --disable-database flag and specifying the --build and --host platforms. The latter are needed to ensure that it knows it is cross-compiling so that it uses the build platform's tool (called tic) to build the terminfo rather than the host's.

The packages bin directory now includes a couple more utilities and a new share directory which is 7.2 MB.

As far as I can tell ncurses is only used in the lua recipe so I don't think the additions here will affect any other recipes. But it would be good to get a second opinion that these changes are OK.

@ianthomas23 ianthomas23 added the enhancement New feature or request label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant