public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] intl: Add test case for bug 16621
@ 2023-09-04 13:33 Florian Weimer
  2023-10-02 19:38 ` Adhemerval Zanella Netto
  0 siblings, 1 reply; 2+ messages in thread
From: Florian Weimer @ 2023-09-04 13:33 UTC (permalink / raw)
  To: libc-alpha

---
 intl/Makefile             |  6 +++++-
 intl/tst-gettext-c-utf8.c | 37 +++++++++++++++++++++++++++++++++++++
 2 files changed, 42 insertions(+), 1 deletion(-)

diff --git a/intl/Makefile b/intl/Makefile
index d7223256eb..a6dd138dad 100644
--- a/intl/Makefile
+++ b/intl/Makefile
@@ -32,7 +32,10 @@ test-srcs := tst-gettext tst-translit tst-gettext2 tst-codeset tst-gettext3
 ifeq ($(have-thread-library),yes)
 test-srcs += $(multithread-test-srcs)
 endif
-tests = tst-ngettext
+tests = \
+  tst-gettext-c-utf8 \
+  tst-ngettext \
+  # tests
 
 before-compile += $(objpfx)msgs.h
 
@@ -117,6 +120,7 @@ $(objpfx)tst-gettext3.out: $(gen-locales)
 $(objpfx)tst-gettext4.out: $(gen-locales)
 $(objpfx)tst-gettext5.out: $(gen-locales)
 $(objpfx)tst-gettext6.out: $(gen-locales)
+$(objpfx)tst-gettext-c-utf8.out: $(gen-locales)
 $(objpfx)tst-translit.out: $(gen-locales)
 endif
 
diff --git a/intl/tst-gettext-c-utf8.c b/intl/tst-gettext-c-utf8.c
new file mode 100644
index 0000000000..bdb9b5396c
--- /dev/null
+++ b/intl/tst-gettext-c-utf8.c
@@ -0,0 +1,37 @@
+/* Check that C.UTF-8 uses C translations (bug 16621).
+   Copyright (C) 2021-2023 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
+
+#include <errno.h>
+#include <stdlib.h>
+#include <string.h>
+#include <support/check.h>
+#include <support/support.h>
+
+static int
+do_test (void)
+{
+  if (setenv ("LANGUAGE", "de_DE.UTF-8", 1) != 0)
+    FAIL_EXIT1 ("setenv");
+  if (setenv ("LC_ALL", "C.UTF-8", 1) != 0)
+    FAIL_EXIT1 ("setenv");
+  xsetlocale (LC_ALL, "");
+  TEST_COMPARE_STRING (strerror (ENOENT), "No such file or directory");
+  return 0;
+}
+
+#include <support/test-driver.c>

base-commit: 2897b231a6b71ee17d47d3d63f1112b2641a476c


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] intl: Add test case for bug 16621
  2023-09-04 13:33 [PATCH] intl: Add test case for bug 16621 Florian Weimer
@ 2023-10-02 19:38 ` Adhemerval Zanella Netto
  0 siblings, 0 replies; 2+ messages in thread
From: Adhemerval Zanella Netto @ 2023-10-02 19:38 UTC (permalink / raw)
  To: Florian Weimer, libc-alpha

LGTM, thanks.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

On 04/09/23 10:33, Florian Weimer via Libc-alpha wrote:
> ---
>  intl/Makefile             |  6 +++++-
>  intl/tst-gettext-c-utf8.c | 37 +++++++++++++++++++++++++++++++++++++
>  2 files changed, 42 insertions(+), 1 deletion(-)
> 
> diff --git a/intl/Makefile b/intl/Makefile
> index d7223256eb..a6dd138dad 100644
> --- a/intl/Makefile
> +++ b/intl/Makefile
> @@ -32,7 +32,10 @@ test-srcs := tst-gettext tst-translit tst-gettext2 tst-codeset tst-gettext3
>  ifeq ($(have-thread-library),yes)
>  test-srcs += $(multithread-test-srcs)
>  endif
> -tests = tst-ngettext
> +tests = \
> +  tst-gettext-c-utf8 \
> +  tst-ngettext \
> +  # tests
>  
>  before-compile += $(objpfx)msgs.h
>  
> @@ -117,6 +120,7 @@ $(objpfx)tst-gettext3.out: $(gen-locales)
>  $(objpfx)tst-gettext4.out: $(gen-locales)
>  $(objpfx)tst-gettext5.out: $(gen-locales)
>  $(objpfx)tst-gettext6.out: $(gen-locales)
> +$(objpfx)tst-gettext-c-utf8.out: $(gen-locales)
>  $(objpfx)tst-translit.out: $(gen-locales)
>  endif
>  
> diff --git a/intl/tst-gettext-c-utf8.c b/intl/tst-gettext-c-utf8.c
> new file mode 100644
> index 0000000000..bdb9b5396c
> --- /dev/null
> +++ b/intl/tst-gettext-c-utf8.c
> @@ -0,0 +1,37 @@
> +/* Check that C.UTF-8 uses C translations (bug 16621).
> +   Copyright (C) 2021-2023 Free Software Foundation, Inc.
> +   This file is part of the GNU C Library.
> +
> +   The GNU C Library is free software; you can redistribute it and/or
> +   modify it under the terms of the GNU Lesser General Public
> +   License as published by the Free Software Foundation; either
> +   version 2.1 of the License, or (at your option) any later version.
> +
> +   The GNU C Library is distributed in the hope that it will be useful,
> +   but WITHOUT ANY WARRANTY; without even the implied warranty of
> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> +   Lesser General Public License for more details.
> +
> +   You should have received a copy of the GNU Lesser General Public
> +   License along with the GNU C Library; if not, see
> +   <https://www.gnu.org/licenses/>.  */
> +
> +#include <errno.h>
> +#include <stdlib.h>
> +#include <string.h>
> +#include <support/check.h>
> +#include <support/support.h>
> +
> +static int
> +do_test (void)
> +{
> +  if (setenv ("LANGUAGE", "de_DE.UTF-8", 1) != 0)
> +    FAIL_EXIT1 ("setenv");
> +  if (setenv ("LC_ALL", "C.UTF-8", 1) != 0)
> +    FAIL_EXIT1 ("setenv");
> +  xsetlocale (LC_ALL, "");
> +  TEST_COMPARE_STRING (strerror (ENOENT), "No such file or directory");
> +  return 0;
> +}
> +
> +#include <support/test-driver.c>
> 
> base-commit: 2897b231a6b71ee17d47d3d63f1112b2641a476c
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-10-02 19:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-04 13:33 [PATCH] intl: Add test case for bug 16621 Florian Weimer
2023-10-02 19:38 ` Adhemerval Zanella Netto

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).