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

mc fails to link to ncurses built as two libraries: ncurses+tinfo. fails as: ./.libs/libinternal.a(tty-ncurses.o): undefined reference to symbol 'meta' /lib64/libtinfow.so.6: error adding symbols: DSO missing from command line #3880

Closed
mc-butler opened this issue Nov 26, 2017 · 9 comments
Assignees
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3880
Reporter slyfox (@trofi)

When ncurses library is built with --with-termlib option
the resulting installation contains two libraries:

/lib64/libncurses.so.6
/lib64/libncursesw.so.6
/lib64/libtinfo.so.6
/lib64/libtinfow.so.6

As a result mc build fails as:

$ ./configure --with-screen=ncurses
$ make

  CCLD     mc
/usr/lib/gcc/x86_64-pc-linux-gnu/5.4.0/../../../../x86_64-pc-linux-gnu/bin/ld: ./.libs/libinternal.a(tty-ncurses.o): undefined reference to symbol 'meta'
/lib64/libtinfow.so.6: error adding symbols: DSO missing from command line

As a workaround gentoo applies the following patch: https://gitweb.gentoo.org/repo/gentoo.git/tree/app-misc/mc/files/mc-4.8.13-tinfo.patch

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Nov 26, 2017 at 22:18 UTC

mc-4.8.13-tinfo.patch

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Nov 27, 2017 at 6:49 UTC (comment 1)

  • Owner set to zaytsev
  • Status changed from new to accepted
  • Milestone changed from Future Releases to 4.8.21

Looks legit, Andrew?

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Nov 27, 2017 at 10:12 UTC (comment 2)

Perhaps yes, but I'd like to test this patch.
And I'd like to make some formatting: add space after comma: sed -e 's|,\[|, [|g'

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2017 at 8:50 UTC (comment 3)

  • Owner changed from zaytsev to andrew_b

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2017 at 8:51 UTC (comment 4)

  • Branch state changed from no branch to on review
  • Votes set to andrew_b

Branch: 3880_fail_ncurses_link
[bf771dc]

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Dec 2, 2017 at 10:09 UTC (comment 5)

  • Votes changed from andrew_b to andrew_b zaytsev
  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2017 at 11:16 UTC (comment 6)

  • Branch state changed from approved to merged
  • Votes changed from andrew_b zaytsev to committed-master

Merged to master: [917cd17].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2017 at 11:17 UTC (comment 7)

  • Status changed from accepted to testing
  • Resolution set to fixed

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 2, 2017 at 11:18 UTC (comment 8)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Development

No branches or pull requests

2 participants