⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@CosmicToast
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-musl
    • aarch64-musl

@CosmicToast
Copy link
Contributor Author

CosmicToast commented Jan 22, 2026

I'll be busy after tonight until probably monday re: the ghostty conflict (so I won't take a closer look today). I'd imagine that since the ghostty term definition only exists on x86_64-glibc, if the definition present in ncurses is the same, the other package can be removed.

Update: the terminal definition is indeed exactly identical. @Duncaen do you think it's ok to remove the terminfo subpackage from ghostty in favor of using the ncurses one? It seems like they've been (relatively) reactive at updating terminfos. Not sure what the alternative would be (I doubt setting ghostty-terminfo conflicting with ncurses is an acceptable resolution).

@Duncaen
Copy link
Member

Duncaen commented Jan 22, 2026

The common solution seems to be to prefer the package provided ones since they always exactly match the terminal.

# resolve conflict with *-terminfo packages
rm ${DESTDIR}/usr/share/terminfo/a/alacritty{,-direct}
rm ${DESTDIR}/usr/share/terminfo/c/contour{,-direct,-latest}
rm ${DESTDIR}/usr/share/terminfo/d/dvtm{,-256color}
rm ${DESTDIR}/usr/share/terminfo/s/st{,-256color}
rm ${DESTDIR}/usr/share/terminfo/f/foot{,-direct,+base}
rm ${DESTDIR}/usr/share/terminfo/r/rio{,-direct}
rm ${DESTDIR}/usr/share/terminfo/w/wezterm

@Duncaen
Copy link
Member

Duncaen commented Jan 22, 2026

Its a bit annoying that as a result all those removed duplicated won't be available by default tho, but that is probably better solved separately (maybe with a meta package that pulls in all terminfo files).

@CosmicToast
Copy link
Contributor Author

Sounds like a good solution - since it's (nowadays) reasonable to expect most terminfos of note to end up in ncurses, it can also be a quick sanity check to see if anything's missing from that metapackage. Force-pushed the added rm. Didn't test it, but there's no reason for it to fail relative to the prior runs.

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