Closed dequbed closed 2 years ago
Thanks for the patch! The context you provided in the pull request comment is essential. Would you please add that context as a comment on the flag in the Cabal file?
Is tinfo/ncurses still a dependency? I don't see it.
@iphydf it isn't a direct dependency. I had forgotten about this, but I looked and it's linked in because it's a dependency of terminfo
. So it seems to me that this linking (platform) issue should be fixed in terminfo
instead.
Hah. Ancient history: https://github.com/judah/terminfo/issues/17
Given my re-read of the situation, I'm going to close this.
Some Linux distributions (such as Gentoo) build ncurses with a split-out tinfo library. On those platforms vty applications will fail to build because symbols like 'set_curterm' are only defined when using '-ltinfo -lcurses' instead of just '-lcurses'. pkg-config will always provide the correct linking options on platforms that have pkg-config installed. All other systems can use the flag 'no-pkgconfig' to cabal configure to prevent cabal calling pkg-config.