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 should be built with --enable-termcap to satisfy libreadline dependencies #3807

Closed
akesandgren opened this issue Nov 16, 2016 · 7 comments
Milestone

Comments

@akesandgren
Copy link
Contributor

akesandgren commented Nov 16, 2016

The easyconfig for libreadline says:

\# for the termcap symbols, use EB ncurses
preconfigopts = "env LDFLAGS='-lncurses'"

But the easybuilt ncurses does not contain, for instance, this termcap symbol, tgetflag, which libreadline uses.

ncurses needs to be built with --enable-termcap to contain the required symbols.
And to work for libreadline, --with-termlib can not be used (like it currently is for ncurses-6.0.eb) becuse then those symbols end up in libtinfo

@boegel boegel added this to the 3.x milestone Nov 20, 2016
@boegel
Copy link
Member

boegel commented Nov 20, 2016

@wpoely86 I think you've looked into this before, thoughts?

I'm not sure if switching to not using --with-termlib is an option now though, unless that's a pure bug fix and doesn't cause any other fallout...

@boegel boegel modified the milestones: 3.1.0, 3.x Nov 20, 2016
@wpoely86
Copy link
Member

Weird, our ncurses do have this symbol. I checked and we don't use libtinfo anywhere in ncurses (or libreadline).

You seem to have an interesting build system @akesandgren What OS are you using to trigger this?

@boegel boegel added the 2018b label Feb 1, 2017
@boegel boegel modified the milestones: 3.1.0, 3.2.0 Feb 1, 2017
@geimer
Copy link
Contributor

geimer commented Feb 23, 2017

Maybe kind of related: Even though ncurses is listed as a build dependency for LLVM, it seems to pick up libtinfo.so from the system, as --with-termlib is not given for the non-dummy ncurses builds. So, if --with-termlib can't be used for libreadline, but may be needed for LLVM, we are in a catch-22 situation, no?

@wpoely86
Copy link
Member

wpoely86 commented Feb 23, 2017

Not really I think.
We need to enable --with-termlib but that means we can drop -lncurses and replace it by -ltinfo in a lot of cases (including libreadline). At least, that's how I understand it.

@akesandgren
Copy link
Contributor Author

@wpoely86 Ubuntu xenial, I had no changes to the ncurses easyconfigs at the time.

@boegel boegel modified the milestones: 3.2.0, 3.3.0 May 2, 2017
@boegel boegel modified the milestones: 3.3.0, 3.4.0 Jun 25, 2017
@boegel
Copy link
Member

boegel commented Sep 6, 2017

@akesandgren Is this still relevant after the changes in #5067 and #5074?

@akesandgren
Copy link
Contributor Author

Hopefully not, but I don't remember what chain was actually hit by the problem.
Let's close this one as not needed due to the above and go on with life.

@boegel boegel closed this as completed Sep 7, 2017
@boegel boegel removed the 2018b label Jun 15, 2018
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

No branches or pull requests

4 participants