Ticket #3880 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

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

Reported by: slyfox Owned by: andrew_b
Priority: major Milestone: 4.8.21
Component: mc-core Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

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

Attachments

mc-4.8.13-tinfo.patch​ (1.7 KB) - added by slyfox 7 years ago.
mc-4.8.13-tinfo.patch

Change History

Changed 7 years ago by slyfox

mc-4.8.13-tinfo.patch

comment:1 Changed 7 years ago by zaytsev

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

Looks legit, Andrew?

comment:2 Changed 7 years ago by andrew_b

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'

comment:3 Changed 7 years ago by andrew_b

  • Owner changed from zaytsev to andrew_b

comment:4 Changed 7 years ago by andrew_b

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

Branch: 3880_fail_ncurses_link
charset:bf771dc8a779c80ffa8e0afdfd4febbe580cb408

Version 0, edited 7 years ago by andrew_b (next)

comment:5 Changed 7 years ago by zaytsev

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

comment:6 Changed 7 years ago by andrew_b

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

Merged to master: [917cd17281cffca7b6819e0c81a787b657ded93c].

comment:7 Changed 7 years ago by andrew_b

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

comment:8 Changed 7 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.