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

Linking to ncurses was not done correctly... #5074

Merged
merged 5 commits into from
Aug 31, 2017

Conversation

ocaisa
Copy link
Member

@ocaisa ocaisa commented Aug 30, 2017

@fgeorgatos
Copy link
Contributor

@ocaisa: ugh, who else gets affected by this? (the old ghosts are waking up now)

@ocaisa
Copy link
Member Author

ocaisa commented Aug 30, 2017

I don't really know, I'm surprised no-one has caught it until now. I only caught it because of the #5067 where the incorrectly linked libreadline was causing the system Lua (and therefore Lmod) to die. Appears to depend on the OS if this makes a difference (I was using Ubuntu 16.10)

@boegel
Copy link
Member

boegel commented Aug 30, 2017

@ocaisa We should make this change consistent across all libreadline easyconfigs?

@boegel boegel added this to the 3.4.0 milestone Aug 30, 2017
@ocaisa
Copy link
Member Author

ocaisa commented Aug 31, 2017

I was afraid you might say that...in this case I just focussed on the latest version of libreadline, the previous one probably also needs a patch (see http://www.linuxfromscratch.org/lfs/view/7.10/chapter06/readline.html )

sync with develop + fix linking to ncurses in libreadline 7.0 easyconfig using GCCcore/6.4.0
@boegel
Copy link
Member

boegel commented Aug 31, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2049.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/c12661eb56f1d2899a8b30efce91501a for a full test report.

@boegel
Copy link
Member

boegel commented Aug 31, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2443.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/784a414d4b9ac2a84ba0cbf9c520e0c8 for a full test report.

@boegel
Copy link
Member

boegel commented Aug 31, 2017

Going in, thanks @ocaisa!

@boegel boegel merged commit e9476c7 into easybuilders:develop Aug 31, 2017
@boegel
Copy link
Member

boegel commented Aug 31, 2017

@ocaisa To get back to applying the same fix consistently to the libreadline 6.3 easyconfigs: the patching mentioned in http://www.linuxfromscratch.org/lfs/view/7.10/chapter06/readline.html is only to avoid renaming the existing library files in case of an in-place update, which is not relevant in the EasyBuild context (since each libreadline installation goes to its own installation prefix).

So probably we can/should also apply the SHLIB_EXT change to older libreadline easyconfigs, like we did for libreadline-6.3.eb in #5067?

Not sure it's worth the effort though, since nobody has complained about problems there for a very long time...

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.

3 participants