nvi @1.81.6_6 build failure: implicit declaration of function 'conv_enc'

[rdprdprdp (Richard Palmer)]

Warning: Configuration logfiles contain indications of -Wimplicit-function-declaration; check that features were not accidentally disabled:

/opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_editors_nvi/nvi/main.log

[rdprdprdp (Richard Palmer)]

Build log of nvi;.

[bK4gYuRo]

'conv_enc' problem can be circumvented (I don't claim 'fixed') by using -Wno-implicit-function-declaration as homebrew does in its formula for nvi in ​https://github.com/Homebrew/homebrew-core/blob/03953b73769b91e3fe2f8bdb8291d42107eb34f6/Formula/nvi.rb

When I got past this problem, I faced another one, with linking.

Apparently, libtool needs this variable to be able to link nvi:

# Flag that allows shared libraries with undefined symbols to be built.
allow_undefined_flag="\${wl}-undefined \${wl}dynamic_lookup"

Unfortunately, configure leaves this variable unset on any macOS above 10.*:

    darwin* | rhapsody*)
      case $host_os in
        rhapsody* | darwin1.[012])
         allow_undefined_flag='${wl}-undefined ${wl}suppress'
         ;;
       *) # Darwin 1.3 on
         if test -z ${MACOSX_DEPLOYMENT_TARGET} ; then
           allow_undefined_flag='${wl}-flat_namespace ${wl}-undefined ${wl}suppress'
         else
           case ${MACOSX_DEPLOYMENT_TARGET} in
             10.[012])
               allow_undefined_flag='${wl}-flat_namespace ${wl}-undefined ${wl}suppress'
               ;;
             10.*)
               allow_undefined_flag='${wl}-undefined ${wl}dynamic_lookup'
               ;;
           esac
         fi
         ;;
      esac

So my crude fix for Ventura was a patch for configure. Build succeeded and I got a usable (in my limited testing) binary.

I am attaching my modified portfile and patch of configure

[bK4gYuRo]

modified portfile

[bK4gYuRo]

patch configure for Ventura

[kencu (Ken)]

Second part of that looks like a slightly different version of the libtool issue we’ve had to patch on a number of ports, eg

​https://github.com/macports/macports-ports/blob/baf9f73fd0e567bd20fd55aed08a643a03f89dd7/x11/blackbox/files/dynamic_lookup-11.patch

[ryandesign (Ryan Carsten Schmidt)]

Replying to bK4gYuRo:

'conv_enc' problem can be circumvented (I don't claim 'fixed') by using -Wno-implicit-function-declaration as homebrew does

These errors need to be fixed properly, not suppressed; see WimplicitFunctionDeclaration.

I'll see if I can do something to fix this port.

[ryandesign (Ryan Carsten Schmidt)]

Replying to kencu:

Second part of that looks like a slightly different version of the libtool issue we’ve had to patch on a number of ports, eg

​https://github.com/macports/macports-ports/blob/baf9f73fd0e567bd20fd55aed08a643a03f89dd7/x11/blackbox/files/dynamic_lookup-11.patch

That's an outdated version of the patch that didn't properly handle the case where the deployment target is not set on PowerPC Mac OS X 10.3 and 10.4 (which, granted, should not occur in MacPorts, but I like using the most correct patch anyway). This is the correct current version:

​https://github.com/macports/macports-ports/blob/16e3be56c3e3ea515bac7fd7f18177063564b03a/archivers/dar/files/dynamic_lookup-11.patch

Read my comments in the bug report whose URL is at the top of that patch if you want to understand its logic.

[ryandesign (Ryan Carsten Schmidt)]

[a0cae35e9fce0f3d591af204ff72aa0a98606d05/macports-ports].