gtypist requires ncursesw

[paxswill]

The recent bump in gnu-typist version (@9d36bab3) now requires ncursesw. It fails during the configure when it can't find the ncursesw header files (which are not installed by default). I also found an issue about ncursesw being removed (#4624). The version of gnu-typist immediately before the version bump (@164c57f2) still compiles, so this seems to be a regression.

Required logs here. Is the way forward re-adding the ncursesw formula, or trying to see if gnu-typist can be patched to get it working with the system ncurses?

[adamv]

@jacknagel Does version 2.9.1 work for you? Doesn't work here. After fixing the ncursesw references, I still get:

Undefined symbols:
  "_iconv_open", referenced from:
      _convertUTF8ToCurrentEncoding in utf8.o
  "_iconv", referenced from:
      _convertUTF8ToCurrentEncoding in utf8.o
  "_iconv_close", referenced from:
      _convertUTF8ToCurrentEncoding in utf8.o

[jacknagel]

Sorry, I didn't build test this one (shame on me). I'll take a look when I get a chance.

[adamv]

Here's my ncurses patch: https://github.com/adamv/homebrew/compare/typo

[jacknagel]

So the system ncurses has wide character symbols, but the library suffix was omitted?

These

http://opensource.apple.com/source/ncurses/ncurses-36/patches.applied/make.diff
http://opensource.apple.com/source/ncurses/ncurses-36/generated.autoconf_configure/config.log

seems to suggest this is the case.

Anyway, I can build it with your patch, it just needs

ENV.append 'LDFLAGS', '-liconv'

[jacobvosmaer]

+1 for the patch with the added ENV.append 'LDFLAGS', '-liconv'

[adamv]

Fixed in de94ec6.