From c9479c29f3ed33b25af573fad98b30acd5652a5a Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Thu, 12 Jan 2023 20:31:31 -0500 Subject: tests: rework gettext to only check 'external' behavior The gettext project deprecated non-external use back in 2010 with the 0.18 release, and made it fatal with the 0.20 release in 2019. With that version, calling AM_GNU_GETTEXT() fails, which means all Automake tests are now skipped. The t/gettext-macros.sh helper probes gettext as such: >+ autopoint --force > autopoint: *** AM_GNU_GETTEXT without 'external' argument is no longer supported in version 0.21.1 > autopoint: *** Stop. > ... >+ aclocal-1.16 -Werror -Wno-syntax -I m4 --install > aclocal-1.16: warnings are treated as errors > configure.ac:4: warning: macro 'AM_GNU_GETTEXT' not found in library > configure.ac:5: warning: macro 'AM_GNU_GETTEXT_VERSION' not found in library >+ echo skip_all_ "couldn't find or get gettext macros" Since t/gettext-macros.sh generates a helper that all other gettext tests use to see if gettext is available, all they get skipped. Rework our existing tests to only check the 'external' gettext mode. This should work with older versions, and we don't really need to keep track of old non-external mode since it's been deprecated for so long. * t/gettext-basics.sh: Use external gettext mode, and adjust tests. * t/gettext-config-rpath.sh: Likewise. * t/gettext-macros.sh: Likewise. * t/gettext-pr381.sh: Likewise. * t/subdir-cond-gettext.sh: Likewise. --- t/gettext-basics.sh | 21 ++++++++------------- t/gettext-config-rpath.sh | 6 +++--- t/gettext-macros.sh | 4 ++-- t/gettext-pr381.sh | 2 +- t/subdir-cond-gettext.sh | 6 +++--- 5 files changed, 17 insertions(+), 22 deletions(-) diff --git a/t/gettext-basics.sh b/t/gettext-basics.sh index 93fe782f4..cb2101086 100644 --- a/t/gettext-basics.sh +++ b/t/gettext-basics.sh @@ -20,7 +20,7 @@ required='gettext' . test-init.sh cat >> configure.ac << 'END' -AM_GNU_GETTEXT +AM_GNU_GETTEXT([external]) AC_OUTPUT END @@ -31,28 +31,23 @@ mkdir po intl $ACLOCAL $AUTOCONF -# po/ and intl/ are required. +# po/ is required. intl/ may not be used with external gettext. +# Internal (bundled) was deprecated upstream in gettext 0.18 (2010) +# and made fatal in gettext 0.20 (2019). AUTOMAKE_fails --add-missing grep 'AM_GNU_GETTEXT.*SUBDIRS' stderr echo 'SUBDIRS = po' >Makefile.am -AUTOMAKE_fails --add-missing -grep 'AM_GNU_GETTEXT.*intl' stderr +# Should not fail. +$AUTOMAKE --add-missing echo 'SUBDIRS = intl' >Makefile.am AUTOMAKE_fails --add-missing grep 'AM_GNU_GETTEXT.*po' stderr -# Ok. - echo 'SUBDIRS = po intl' >Makefile.am -$AUTOMAKE --add-missing - -# Make sure distcheck runs './configure --with-included-gettext'. -./configure -echo distdir: > po/Makefile -echo distdir: > intl/Makefile -$MAKE -n distcheck | grep '.*--with-included-gettext' +AUTOMAKE_fails --add-missing +grep 'intl.*SUBDIRS.*AM_GNU_GETTEXT' stderr : diff --git a/t/gettext-config-rpath.sh b/t/gettext-config-rpath.sh index 3953a0c3d..522b42fce 100644 --- a/t/gettext-config-rpath.sh +++ b/t/gettext-config-rpath.sh @@ -20,14 +20,14 @@ required='gettext' . test-init.sh cat >> configure.ac << 'END' -AM_GNU_GETTEXT +AM_GNU_GETTEXT([external]) # config.rpath is required by versions >= 0.14.3. AM_GNU_GETTEXT_VERSION([0.14.3]) AC_OUTPUT END -echo 'SUBDIRS = po intl' >Makefile.am -mkdir po intl +echo 'SUBDIRS = po' >Makefile.am +mkdir po # If aclocal fails here, it may be that gettext is too old to provide # AM_GNU_GETTEXT_VERSION. Similarly, autopoint will fail if it's diff --git a/t/gettext-macros.sh b/t/gettext-macros.sh index e352e6790..1376be81e 100644 --- a/t/gettext-macros.sh +++ b/t/gettext-macros.sh @@ -47,8 +47,8 @@ autopoint_version=$(extract_program_version autopoint) \ cat > configure.ac <> configure.ac << 'END' -AM_GNU_GETTEXT +AM_GNU_GETTEXT([external]) AC_OUTPUT END diff --git a/t/subdir-cond-gettext.sh b/t/subdir-cond-gettext.sh index 023eff134..a203c025d 100644 --- a/t/subdir-cond-gettext.sh +++ b/t/subdir-cond-gettext.sh @@ -20,18 +20,18 @@ required=gettext . test-init.sh cat >> configure.ac << 'END' -AM_GNU_GETTEXT +AM_GNU_GETTEXT([external]) AM_CONDITIONAL([MAUDE], [true]) ALL_LINGUAS= AC_SUBST([ALL_LINGUAS]) END -mkdir po intl +mkdir po : >config.rpath cat > Makefile.am << 'END' if MAUDE -SUBDIRS = po intl +SUBDIRS = po else SUBDIRS = endif -- cgit v1.2.1