Re: textinfo-6.8: Test failing

[Gavin Smith]

On Sun, Jul 4, 2021 at 7:04 PM Chris Clayton <chris2553@googlemail.com> wrote:
> Thanks for the reply, Gavin.
>
> Right, thanks. the two commands give:
>
> $ locale
> LANG=en_GB.UTF-8
> LC_CTYPE="en_GB.UTF-8"
> LC_NUMERIC="en_GB.UTF-8"
> LC_TIME="en_GB.UTF-8"
> LC_COLLATE="en_GB.UTF-8"
> LC_MONETARY="en_GB.UTF-8"
> LC_MESSAGES="en_GB.UTF-8"
> LC_PAPER="en_GB.UTF-8"
> LC_NAME="en_GB.UTF-8"
> LC_ADDRESS="en_GB.UTF-8"
> LC_TELEPHONE="en_GB.UTF-8"
> LC_MEASUREMENT="en_GB.UTF-8"
> LC_IDENTIFICATION="en_GB.UTF-8"
> LC_ALL=
>
> $ locale -a
> C
> en_GB
> en_GB.iso88591
> en_GB.utf8
> en_GB.UTF-8
> en_US.utf8
> POSIX
>
> I've just built texinfo-6.7 and the test suite succeeds.  I'll investigate 
> what changed in that test. Thanks.

This isn't what I expected as you have a locale "en_US.utf8" which I
understand is the same thing as "en_US.UTF-8", which is one of the
locales tried in 'gdt' in tp/Texinfo/Report.pm. You could try the
following change:-

diff --git a/tp/Texinfo/Report.pm b/tp/Texinfo/Report.pm
index d72b5ba7ba..1a30a50f23 100644
--- a/tp/Texinfo/Report.pm
+++ b/tp/Texinfo/Report.pm
@@ -251,7 +251,7 @@ sub gdt($$;$$)
   # This may not work if a locale named here doesn't exist on the system.
   # Set LC_ALL rather than LC_MESSAGES as LC_MESSAGES may not be supported
   # on Perl for MS-Windows.
-  for my $try ('en_US.UTF-8', 'en_US') {
+  for my $try ('en_US.utf8', 'en_US') {
     my $locale = POSIX::setlocale(LC_ALL, $try);
     last if $locale;
   }

- to see if it makes a difference. If it is this part of the code that
is causing the test failure then it would be due to upgrading the
bundled version of libintl-perl, due to a fix for the issue at
https://rt.cpan.org/Public/Bug/Display.html?id=81315

I reported the issue here
https://lists.gnu.org/archive/html/bug-texinfo/2020-11/msg00088.html -
but nobody seemed to have an idea of what should be done.