[pkgsrc/trunk]: pkgsrc/devel/ncurses ncurses: Simplify handling of TOOLS_TBL.

[jperkin <jperkin%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/410bc05bcd09
branches:  trunk
changeset: 433277:410bc05bcd09
user:      jperkin <jperkin%pkgsrc.org@localhost>
date:      Mon Jun 01 07:01:19 2020 +0000

description:
ncurses: Simplify handling of TOOLS_TBL.

Don't try to handle obscure cases where certain parts of curses are builtin,
historically it has been shown to lead to failures and a growing list of
special cases.  Just disable completely if native tbl(1) is not available.

Fixes latest issue seen by sjmulder on OpenIndiana.

diffstat:

 devel/ncurses/Makefile.common |  21 ++++++---------------
 1 files changed, 6 insertions(+), 15 deletions(-)

diffs (44 lines):

diff -r ccc655704338 -r 410bc05bcd09 devel/ncurses/Makefile.common
--- a/devel/ncurses/Makefile.common     Mon Jun 01 06:28:49 2020 +0000
+++ b/devel/ncurses/Makefile.common     Mon Jun 01 07:01:19 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile.common,v 1.45 2020/03/09 06:11:55 wiz Exp $
+# $NetBSD: Makefile.common,v 1.46 2020/06/01 07:01:19 jperkin Exp $
 #
 # used by devel/ncurses/Makefile
 # used by devel/ncursesw/Makefile
@@ -27,7 +27,6 @@
 CONFIGURE_ARGS+=       --without-ada
 CONFIGURE_ARGS+=       --without-gpm
 CONFIGURE_ARGS+=       --with-manpage-format=normal
-CONFIGURE_ARGS+=       --with-manpage-tbl
 CONFIGURE_ARGS+=       --without-manpage-renames
 CONFIGURE_ARGS+=       --with-default-terminfo-dir=${PREFIX}/${TERMINFODIR}
 CONFIGURE_ARGS+=       --with-terminfo-dirs=${PREFIX}/${TERMINFODIR}
@@ -49,21 +48,13 @@
 CONFIGURE_ENV+=                ac_cv_path_TIC_PATH=${LOCALBASE}/bin/tic
 .endif
 
-.if !defined(TOOLS_TBL)
-# This means that groff needs to be built to get tbl(1).
-# We can only build groff if there is a builtin implementation of curses,
-# otherwise we get a circular dependency problem.
-# Additionally, Interix is special, see ../../mk/curses.buildlink3.mk
-# Additionally, Arch Linux boostrap comes without curses package.
-.  if !(exists(/usr/include/curses.h) || exists(/usr/include/ncurses.h)) || \
-       ${OPSYS} == "Interix" || \
-       (${OPSYS} == "FreeBSD" && empty(OS_VERSION:M[0-9].*) && \
-               empty(OS_VERSION:M1[01].*)) || \
-       (${OPSYS} == "Linux" && exists(/etc/arch-release))
-# None was available, so disable use of tbl(1)
+# If there's no native tbl(1) then the tools infrastructure would pull in
+# groff, causing various circular dependencies.
+.if defined(TOOLS_TBL)
+CONFIGURE_ARGS+=       --with-manpage-tbl
+.else
 USE_TOOLS:=            ${USE_TOOLS:Ntbl}
 CONFIGURE_ARGS+=       --without-manpage-tbl
-.  endif
 .endif
 
 .include "../../mk/compiler.mk"