[pkgsrc/trunk]: pkgsrc/devel/gettext-lib gettext-lib: declare built-in implem...

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

details:   https://anonhg.NetBSD.org/pkgsrc/rev/4dc35fa28986
branches:  trunk
changeset: 403670:4dc35fa28986
user:      triaxx <triaxx%pkgsrc.org@localhost>
date:      Wed Oct 30 14:43:20 2019 +0000

description:
gettext-lib: declare built-in implementation if ngettext() supplied

The H_NGETTEXT_GETTEXT symbol was tested to determine whether we should use
the built-in implementation and to declare USE_BUILTIN.gettext. According
the associated comment, this symbol should be tested to determine if there
is a built-in implementation and to declare IS_BUILTIN.gettext.

diffstat:

 devel/gettext-lib/builtin.mk |  16 +++++++---------
 1 files changed, 7 insertions(+), 9 deletions(-)

diffs (43 lines):

diff -r ec7ffd386f83 -r 4dc35fa28986 devel/gettext-lib/builtin.mk
--- a/devel/gettext-lib/builtin.mk      Wed Oct 30 14:16:19 2019 +0000
+++ b/devel/gettext-lib/builtin.mk      Wed Oct 30 14:43:20 2019 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: builtin.mk,v 1.46 2014/07/03 14:59:55 wiz Exp $
+# $NetBSD: builtin.mk,v 1.47 2019/10/30 14:43:20 triaxx Exp $
 
 .include "../../mk/bsd.fast.prefs.mk"
 
@@ -32,12 +32,17 @@
 # SCO OpenServer 5.0.7/3.2 has an unusual scheme where /usr/include/libintl.h
 # pulls in /usr/include/libgnuintl.h, where the latter is the real libintl.h.
 #
+# By default, assume that the native gettext implementation is good
+# enough to replace GNU gettext if it supplies ngettext().
+#
 .if !defined(IS_BUILTIN.gettext)
 IS_BUILTIN.gettext=    no
 .  if (empty(H_GETTEXT:M__nonexistent__) && \
        empty(H_GETTEXT:M${LOCALBASE}/*)) || \
       (empty(H_GENTOO_GETTEXT:M__nonexistent__) && \
        empty(H_GENTOO_GETTEXT:M${LOCALBASE}/*)) || \
+      (empty(H_NGETTEXT_GETTEXT:M__nonexistent__) && \
+       empty(H_NGETTEXT_GETTEXT:M${LOCALBASE}/*)) || \
       (empty(H_OPNSVR5_GETTEXT:M__nonexistent__) && \
        empty(H_OPNSVR5_GETTEXT:M${LOCALBASE}/*))
 IS_BUILTIN.gettext=    yes
@@ -68,14 +73,7 @@
 .        endif
 .      endfor
 .    endif
-# XXX
-# XXX By default, assume that the native gettext implementation is good
-# XXX enough to replace GNU gettext if it supplies ngettext().
-# XXX
-.    if empty(H_NGETTEXT_GETTEXT:M__nonexistent__) && \
-       empty(H_NGETTEXT_GETTEXT:M${LOCALBASE}/*)
-USE_BUILTIN.gettext=   yes
-.    endif
+#
 #
 # Some platforms don't have a gettext implementation that can replace
 # GNU gettext.