This is the mail archive of the
glibc-cvs@sourceware.org
mailing list for the glibc project.
GNU C Library master sources branch master updated. glibc-2.26.9000-666-g02eec68
- From: carlos at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 25 Oct 2017 20:37:11 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.26.9000-666-g02eec68
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via 02eec681676c5aabf2eb13b92b1124245d19112f (commit)
via 56fa555a834c1536bf8d58c1ac6097f18f0d92b6 (commit)
from 337ff3c501f0e1fadd1036b6fa2754cfbb0c29ea (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=02eec681676c5aabf2eb13b92b1124245d19112f
commit 02eec681676c5aabf2eb13b92b1124245d19112f
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Tue Oct 17 01:33:42 2017 -0700
localedef: Add --no-warnings/--warnings option
From localedef --help:
Output control:
...
--no-warnings=<warnings> Comma-separated list of warnings to disable;
supported warnings are: ascii, intcurrsym
...
--warnings=<warnings> Comma-separated list of warnings to enable;
supported warnings are: ascii, intcurrsym
Locales using SHIFT_JIS and SHIFT_JISX0213 character maps are not ASCII
compatible. In order to build locales using these character maps, and
have localedef exit with a status of 0, we add new option to localedef
to disable or enable specific warnings. The options are --no-warnings
and --warnings, to disable and enable specific warnings respectively.
The options take a comma-separated list of warning names. The warning
names are taken directly from the generated warning. When a warning
that can be disabled is issued it will print something like this: foo is
not defined [--no-warnings=foo]
For the initial implementation we add two controllable warnings; first
'ascii' which is used by the localedata installation makefile target to
install SHIFT_JIS and SHIFT_JISX0213-using locales without error; second
'intcurrsym' which allows a program to use a non-standard international
currency symbol without triggering a warning. The 'intcurrsym' is
useful in the future if country codes are added that are not in our
current ISO 4217 list, and the user wants to avoid the warning. Having
at least two warnings to control gives an example for how the changes
can be extended to more warnings if required in the future.
These changes allow ja_JP.SHIFT_JIS and ja_JP.SHIFT_JISX0213 to be
compiled without warnings using --no-warnings=ascii. The
localedata/Makefile $(INSTALL-SUPPORTED-LOCALES) target is adjusted to
automatically add `--no-warnings=ascii` for such charmaps, and likewise
localedata/gen-locale.sh is adjusted with similar logic.
v2: Bring verbose, be_quiet, and all warning control booleans into
record-status.c, and compile this object file to be used by locale,
iconv, and localedef. Any users include record-status.h.
v3: Fix an instance of boolean coercion in set_warning().
Signed-off-by: Carlos O'Donell <carlos@redhat.com>
diff --git a/ChangeLog b/ChangeLog
index 9ae9618..fe715d9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,34 @@
2017-10-25 Carlos O'Donell <carlos@redhat.com>
+ * locale/programs/record-status.h: Define globals, and function
+ prototypes. Move function bodies...
+ * locale/programs/record-status.c: ... to here. New file.
+ * iconv/Makefile (iconv_prog-modules): Add record-status.
+ * locale/Makefile (lib-modules): Likewise.
+ * iconv/iconv_prog.c: Remove verbose.
+ * iconv/iconv_prog.h: Include record-status.h (defines verbose).
+ * locale/programs/charmap.c (charmap_read): If warn_ascii is true then
+ record a warning about ASCII compatibility.
+ * locale/programs/ld-monetary.c (monetary_finish): If
+ warn_int_curr_symbol is true then record a warning about the symbol
+ not being in our ISO 4217 list.
+ * locale/programs/locale.c: Include record-status.h. Remove verbose.
+ * locale/programs/localedef.c: Include ctype.h. Remove delcaration of
+ verbose, recorded_warning_count, recorded_error_count, and be_quiet.
+ (OPT_NO_WARN): Define.
+ (OPT_WARN): Define.
+ (options): Add entry for --no-warnings, and --warnings.
+ (set_warnings): New function to enable/disable warnings.
+ (parse_opt): Call set_warnings for OPT_NO_WARN and OPT_WARN.
+ * locale/programs/localedef.h: Remove warn_int_curr_symbol.
+ * localedata/gen-locale.sh: Default flags to `--quiet -c'.
+ Add `--no-warnings=ascii' to locales using SHIFT_JIS or SHIFT_JIXX0213.
+ Pass flags to generate_locale.
+ (generate_locale): Accept new flag argument and pass it to localedef
+ invocation.
+ * localedata/Makefile (INSTALL-SUPPORTED-LOCALES): Use
+ --no-warnings=ascii for SHIFT_JIS and SHIFT_JISX0213 charmaps.
+
* localedata/Makefile (test-input-data): Use full file name.
* localedata/da_DK.in: Rename to...
* localedata/da_DK.ISO-8859-1.in: ...this.
diff --git a/iconv/Makefile b/iconv/Makefile
index fd35751..d340565 100644
--- a/iconv/Makefile
+++ b/iconv/Makefile
@@ -31,7 +31,8 @@ routines += gconv_dl
vpath %.c ../locale/programs ../intl
iconv_prog-modules = iconv_charmap charmap charmap-dir linereader \
- dummy-repertoire simple-hash xstrdup xmalloc
+ dummy-repertoire simple-hash xstrdup xmalloc \
+ record-status
iconvconfig-modules = strtab xmalloc hash-string
extra-objs = $(iconv_prog-modules:=.o) $(iconvconfig-modules:=.o)
CFLAGS-iconv_prog.c = -I../locale/programs
diff --git a/iconv/iconv_prog.c b/iconv/iconv_prog.c
index 35d6eb3..52b9c78 100644
--- a/iconv/iconv_prog.c
+++ b/iconv/iconv_prog.c
@@ -97,9 +97,6 @@ static const char *to_code = "";
/* File to write output to. If NULL write to stdout. */
static const char *output_file;
-/* Nonzero if verbose ouput is wanted. */
-int verbose;
-
/* Nonzero if list of all coded character sets is wanted. */
static int list;
diff --git a/iconv/iconv_prog.h b/iconv/iconv_prog.h
index 1571fc9..cc2a677 100644
--- a/iconv/iconv_prog.h
+++ b/iconv/iconv_prog.h
@@ -20,11 +20,9 @@
#define _ICONV_PROG_H 1
#include <stdio.h>
-#include <charmap.h>
-
-/* Nonzero if verbose ouput is wanted. */
-extern int verbose;
+#include <charmap.h>
+#include <record-status.h>
/* If nonzero omit invalid character from output. */
extern int omit_invalid;
diff --git a/locale/Makefile b/locale/Makefile
index 98ee762..ddbaad7 100644
--- a/locale/Makefile
+++ b/locale/Makefile
@@ -53,7 +53,8 @@ localedef-modules := localedef $(categories:%=ld-%) \
repertoire locarchive
localedef-aux := md5
locale-modules := locale locale-spec
-lib-modules := charmap-dir simple-hash xmalloc xstrdup
+lib-modules := charmap-dir simple-hash xmalloc xstrdup \
+ record-status
GPERF = gperf
diff --git a/locale/programs/charmap.c b/locale/programs/charmap.c
index a670db9..964d932 100644
--- a/locale/programs/charmap.c
+++ b/locale/programs/charmap.c
@@ -256,9 +256,15 @@ charmap_read (const char *filename, int verbose, int error_not_found,
if (failed)
{
- record_warning (_("\
-character map `%s' is not ASCII compatible, locale not ISO C compliant\n"),
- result->code_set_name);
+ /* A user may disable the ASCII compatibility warning check,
+ but we must remember that the encoding is not ASCII
+ compatible, since it may have other implications. Later
+ we will set _NL_CTYPE_MAP_TO_NONASCII from this value. */
+ if (warn_ascii)
+ record_warning (_(
+"character map `%s' is not ASCII compatible, locale not ISO C compliant "
+"[--no-warnings=ascii]"),
+ result->code_set_name);
enc_not_ascii_compatible = true;
}
}
diff --git a/locale/programs/ld-monetary.c b/locale/programs/ld-monetary.c
index 9d94738..13c9697 100644
--- a/locale/programs/ld-monetary.c
+++ b/locale/programs/ld-monetary.c
@@ -234,12 +234,17 @@ No definition for %s category found"), "LC_MONETARY");
char symbol[4];
strncpy (symbol, monetary->int_curr_symbol, 3);
symbol[3] = '\0';
+ /* A user may disable this waning for testing purposes or
+ for building a locale with a 3 letter country code that
+ was not yet supported in our ISO 4217 list.
+ See the use of --no-warnings=intcurrsym. */
if (bsearch (symbol, valid_int_curr, NR_VALID_INT_CURR,
sizeof (const char *),
- (comparison_fn_t) curr_strcmp) == NULL)
+ (comparison_fn_t) curr_strcmp) == NULL
+ && warn_int_curr_symbol)
record_warning (_("\
%s: value of field `int_curr_symbol' does \
-not correspond to a valid name in ISO 4217"),
+not correspond to a valid name in ISO 4217 [--no-warnings=intcurrsym]"),
"LC_MONETARY");
}
}
diff --git a/locale/programs/locale.c b/locale/programs/locale.c
index a29a32b..939214d 100644
--- a/locale/programs/locale.c
+++ b/locale/programs/locale.c
@@ -40,6 +40,7 @@
#include <sys/mman.h>
#include <sys/stat.h>
+#include "record-status.h"
#include "localeinfo.h"
#include "charmap-dir.h"
#include "../locarchive.h"
@@ -59,13 +60,6 @@ static int do_all;
/* Print names of all available character maps. */
static int do_charmaps = 0;
-/* Nonzero if verbose output is wanted. Note that this definition is
- file-local in scope, and does not extended to uses of verbose in
- record-status.h functions like record_verbose. This means that this
- verbose will not enable record_verbose messages for uses from locale,
- but it does for uses from localdef (where verbose is global). */
-static int verbose;
-
/* Name and version of program. */
static void print_version (FILE *stream, struct argp_state *state);
void (*argp_program_version_hook) (FILE *, struct argp_state *) = print_version;
diff --git a/locale/programs/localedef.c b/locale/programs/localedef.c
index 7d76154..973bb55 100644
--- a/locale/programs/localedef.c
+++ b/locale/programs/localedef.c
@@ -32,6 +32,7 @@
#include <error.h>
#include <sys/mman.h>
#include <sys/stat.h>
+#include <ctype.h>
#include "localedef.h"
#include "charmap.h"
@@ -48,18 +49,6 @@ struct copy_def_list_t *copy_list;
/* If this is defined be POSIX conform. */
int posix_conformance;
-/* If not zero give a lot more messages. */
-int verbose;
-
-/* Warnings recorded by record_warnings (see localedef.h). */
-int recorded_warning_count;
-
-/* Errors recorded by record_error (see localedef.h). */
-int recorded_error_count;
-
-/* If not zero suppress warnings and information messages. */
-int be_quiet;
-
/* If not zero force output even if warning were issued. */
static int force_output;
@@ -114,6 +103,8 @@ void (*argp_program_version_hook) (FILE *, struct argp_state *) = print_version;
#define OPT_LIST_ARCHIVE 309
#define OPT_LITTLE_ENDIAN 400
#define OPT_BIG_ENDIAN 401
+#define OPT_NO_WARN 402
+#define OPT_WARN 403
/* Definitions of arguments for argp functions. */
static const struct argp_option options[] =
@@ -134,6 +125,13 @@ static const struct argp_option options[] =
{ "quiet", OPT_QUIET, NULL, 0,
N_("Suppress warnings and information messages") },
{ "verbose", 'v', NULL, 0, N_("Print more messages") },
+ { "no-warnings", OPT_NO_WARN, N_("<warnings>"), 0,
+ N_("Comma-separated list of warnings to disable; "
+ "supported warnings are: ascii, intcurrsym") },
+ { "warnings", OPT_WARN, N_("<warnings>"), 0,
+ N_("Comma-separated list of warnings to enable; "
+ "supported warnings are: ascii, intcurrsym") },
+
{ NULL, 0, NULL, 0, N_("Archive control:") },
{ "no-archive", OPT_NO_ARCHIVE, NULL, 0,
N_("Don't add new data to archive") },
@@ -309,6 +307,43 @@ no output file produced because errors were issued"));
exit (recorded_warning_count != 0);
}
+/* Search warnings for matching warnings and if found enable those
+ warnings if ENABLED is true, otherwise disable the warnings. */
+static void
+set_warnings (char *warnings, bool enabled)
+{
+ char *tok = warnings;
+ char *copy = (char *) malloc (strlen (warnings) + 1);
+ char *save = copy;
+
+ /* As we make a copy of the warnings list we remove all spaces from
+ the warnings list to make the processing a more robust. We don't
+ support spaces in a warning name. */
+ do
+ {
+ while (isspace (*tok) != 0)
+ tok++;
+ }
+ while ((*save++ = *tok++) != '\0');
+
+ warnings = copy;
+
+ /* Tokenize the input list of warnings to set, compare them to
+ known warnings, and set the warning. We purposely ignore unknown
+ warnings, and are thus forward compatible, users can attempt to
+ disable whaterver new warnings they know about, but we will only
+ disable those *we* known about. */
+ while ((tok = strtok_r (warnings, ",", &save)) != NULL)
+ {
+ warnings = NULL;
+ if (strcmp (tok, "ascii") == 0)
+ warn_ascii = enabled;
+ else if (strcmp (tok, "intcurrsym") == 0)
+ warn_int_curr_symbol = enabled;
+ }
+
+ free (copy);
+}
/* Handle program arguments. */
static error_t
@@ -346,6 +381,14 @@ parse_opt (int key, char *arg, struct argp_state *state)
case OPT_BIG_ENDIAN:
set_big_endian (true);
break;
+ case OPT_NO_WARN:
+ /* Disable the warnings. */
+ set_warnings (arg, false);
+ break;
+ case OPT_WARN:
+ /* Enable the warnings. */
+ set_warnings (arg, true);
+ break;
case 'c':
force_output = 1;
break;
diff --git a/locale/programs/localedef.h b/locale/programs/localedef.h
index 96aa696..89b3ff0 100644
--- a/locale/programs/localedef.h
+++ b/locale/programs/localedef.h
@@ -114,7 +114,6 @@ struct localedef_t
/* Global variables of the localedef program. */
-extern int verbose;
extern const char *repertoire_global;
extern int max_locarchive_open_retry;
extern bool no_archive;
diff --git a/locale/programs/record-status.h b/locale/programs/record-status.c
similarity index 84%
copy from locale/programs/record-status.h
copy to locale/programs/record-status.c
index b6bc58c..ab85d76 100644
--- a/locale/programs/record-status.h
+++ b/locale/programs/record-status.c
@@ -1,4 +1,4 @@
-/* General definitions for recording error and warning status.
+/* Functions for recorded errors, warnings, and verbose messages.
Copyright (C) 1998-2017 Free Software Foundation, Inc.
This file is part of the GNU C Library.
@@ -15,35 +15,39 @@
You should have received a copy of the GNU General Public License
along with this program; if not, see <http://www.gnu.org/licenses/>. */
-#ifndef _RECORD_STATUS_H
-#define _RECORD_STATUS_H 1
-
+#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
+#include <stdbool.h>
+#include <string.h>
#include <error.h>
+#include <errno.h>
#include <locale.h>
-#include <string.h>
-/* We tentatively define all of the global data we use:
- * recorded_warning_count: Number of warnings counted.
- * recorded_error_count: Number of errors counted.
- * be_quiet: Should all calls be silent?
- * verbose: Should verbose messages be printed? */
+#include "record-status.h"
+
+/* Warnings recorded by record_warnings. */
int recorded_warning_count;
+
+/* Errors recorded by record_errors. */
int recorded_error_count;
+
+/* If not zero suppress warnings and information messages. */
int be_quiet;
+
+/* If not zero give a lot more messages. */
int verbose;
-/* Saved locale state. */
-struct locale_state
-{
- /* The current in-use locale. */
- char *cur_locale;
-};
+/* Warnings which can be disabled: */
+/* By default we check the character map for ASCII compatibility. */
+bool warn_ascii = true;
+/* By default we check that the international currency symbol matches a
+ known country code. */
+bool warn_int_curr_symbol = true;
/* Alter the current locale to match the locale configured by the
user, and return the previous saved state. */
-static struct locale_state
+struct locale_state
push_locale (void)
{
int saved_errno;
@@ -70,7 +74,7 @@ push_locale (void)
}
/* Use the saved state to restore the locale. */
-static void
+void
pop_locale (struct locale_state ls)
{
const char *set = NULL;
@@ -89,7 +93,7 @@ pop_locale (struct locale_state ls)
/* Wrapper to print verbose informative messages.
Verbose messages are only printed if --verbose
is in effect and --quiet is not. */
-static void
+void
__attribute__ ((__format__ (__printf__, 2, 3), nonnull (1, 2), unused))
record_verbose (FILE *stream, const char *format, ...)
{
@@ -114,7 +118,7 @@ record_verbose (FILE *stream, const char *format, ...)
pop_locale (ls);
va_end (arg);
- fprintf (stream, "%s\n", str);
+ fprintf (stream, "[verbose] %s\n", str);
free (str);
}
@@ -124,7 +128,7 @@ record_verbose (FILE *stream, const char *format, ...)
many were called because this effects our exit code.
Nothing is printed if --quiet is in effect, but warnings
are always counted. */
-static void
+void
__attribute__ ((__format__ (__printf__, 1, 2), nonnull (1), unused))
record_warning (const char *format, ...)
{
@@ -148,7 +152,7 @@ record_warning (const char *format, ...)
pop_locale (ls);
va_end (arg);
- fprintf (stderr, "%s\n", str);
+ fprintf (stderr, "[warning] %s\n", str);
free (str);
}
@@ -159,7 +163,7 @@ record_warning (const char *format, ...)
Nothing is printed if --quiet is in effect, but errors
are always counted, and fatal errors always exit the
program. */
-static void
+void
__attribute__ ((__format__ (__printf__, 3, 4), nonnull (3), unused))
record_error (int status, int errnum, const char *format, ...)
{
@@ -185,13 +189,13 @@ record_error (int status, int errnum, const char *format, ...)
pop_locale (ls);
va_end (arg);
- error (status, errnum, "%s", str);
+ error (status, errnum, "[error] %s", str);
free (str);
}
}
/* ... likewise for error_at_line. */
-static void
+void
__attribute__ ((__format__ (__printf__, 5, 6), nonnull (3, 5), unused))
record_error_at_line (int status, int errnum, const char *filename,
unsigned int linenum, const char *format, ...)
@@ -218,10 +222,8 @@ record_error_at_line (int status, int errnum, const char *filename,
pop_locale (ls);
va_end (arg);
- error_at_line (status, errnum, filename, linenum, "%s", str);
+ error_at_line (status, errnum, filename, linenum, "[error] %s", str);
free (str);
}
}
-
-#endif
diff --git a/locale/programs/record-status.h b/locale/programs/record-status.h
index b6bc58c..345f7f4 100644
--- a/locale/programs/record-status.h
+++ b/locale/programs/record-status.h
@@ -18,210 +18,34 @@
#ifndef _RECORD_STATUS_H
#define _RECORD_STATUS_H 1
-#include <stdlib.h>
-#include <stdarg.h>
-#include <error.h>
-#include <locale.h>
-#include <string.h>
-
-/* We tentatively define all of the global data we use:
- * recorded_warning_count: Number of warnings counted.
- * recorded_error_count: Number of errors counted.
- * be_quiet: Should all calls be silent?
- * verbose: Should verbose messages be printed? */
-int recorded_warning_count;
-int recorded_error_count;
-int be_quiet;
-int verbose;
-
-/* Saved locale state. */
+#include <stdio.h>
+#include <stdbool.h>
+
+/* Error, warning and verbose count and control. */
+extern int recorded_warning_count;
+extern int recorded_error_count;
+extern int be_quiet;
+extern int verbose;
+extern bool warn_ascii;
+extern bool warn_int_curr_symbol;
+
+/* Record verbose, warnings, or errors... */
+void record_verbose (FILE *stream, const char *format, ...);
+void record_warning (const char *format, ...);
+void record_error (int status, int errnum, const char *format, ...);
+void record_error_at_line (int status, int errnum,
+ const char *filename, unsigned int linenum,
+ const char *format, ...);
+
+/* Locale related functionality for custom error functions. */
struct locale_state
{
/* The current in-use locale. */
char *cur_locale;
};
-/* Alter the current locale to match the locale configured by the
- user, and return the previous saved state. */
-static struct locale_state
-push_locale (void)
-{
- int saved_errno;
- const char *orig;
- char *copy = NULL;
-
- saved_errno = errno;
-
- orig = setlocale (LC_CTYPE, NULL);
- if (orig == NULL)
- error (0, 0, "failed to read locale!");
-
- if (setlocale (LC_CTYPE, "") == NULL)
- error (0, 0, "failed to set locale!");
-
- errno = saved_errno;
-
- if (orig != NULL)
- copy = strdup (orig);
-
- /* We will return either a valid locale or NULL if we failed
- to save the locale. */
- return (struct locale_state) { .cur_locale = copy };
-}
-
-/* Use the saved state to restore the locale. */
-static void
-pop_locale (struct locale_state ls)
-{
- const char *set = NULL;
- /* We might have failed to save the locale, so only attempt to
- restore a validly saved non-NULL locale. */
- if (ls.cur_locale != NULL)
- {
- set = setlocale (LC_CTYPE, ls.cur_locale);
- if (set == NULL)
- error (0, 0, "failed to restore %s locale!", ls.cur_locale);
-
- free (ls.cur_locale);
- }
-}
-
-/* Wrapper to print verbose informative messages.
- Verbose messages are only printed if --verbose
- is in effect and --quiet is not. */
-static void
-__attribute__ ((__format__ (__printf__, 2, 3), nonnull (1, 2), unused))
-record_verbose (FILE *stream, const char *format, ...)
-{
- char *str;
- va_list arg;
-
- if (!verbose)
- return;
-
- if (!be_quiet)
- {
- struct locale_state ls;
- int ret;
-
- va_start (arg, format);
- ls = push_locale ();
-
- ret = vasprintf (&str, format, arg);
- if (ret == -1)
- abort ();
-
- pop_locale (ls);
- va_end (arg);
-
- fprintf (stream, "%s\n", str);
-
- free (str);
- }
-}
-
-/* Wrapper to print warning messages. We keep track of how
- many were called because this effects our exit code.
- Nothing is printed if --quiet is in effect, but warnings
- are always counted. */
-static void
-__attribute__ ((__format__ (__printf__, 1, 2), nonnull (1), unused))
-record_warning (const char *format, ...)
-{
- char *str;
- va_list arg;
-
- recorded_warning_count++;
-
- if (!be_quiet)
- {
- struct locale_state ls;
- int ret;
-
- va_start (arg, format);
- ls = push_locale ();
-
- ret = vasprintf (&str, format, arg);
- if (ret == -1)
- abort ();
-
- pop_locale (ls);
- va_end (arg);
-
- fprintf (stderr, "%s\n", str);
-
- free (str);
- }
-}
-
-/* Wrapper to print error messages. We keep track of how
- many were called because this effects our exit code.
- Nothing is printed if --quiet is in effect, but errors
- are always counted, and fatal errors always exit the
- program. */
-static void
-__attribute__ ((__format__ (__printf__, 3, 4), nonnull (3), unused))
-record_error (int status, int errnum, const char *format, ...)
-{
- char *str;
- va_list arg;
-
- recorded_error_count++;
-
- /* The existing behaviour is that even if you use --quiet, a fatal
- error is always printed and terminates the process. */
- if (!be_quiet || status != 0)
- {
- struct locale_state ls;
- int ret;
-
- va_start (arg, format);
- ls = push_locale ();
-
- ret = vasprintf (&str, format, arg);
- if (ret == -1)
- abort ();
-
- pop_locale (ls);
- va_end (arg);
-
- error (status, errnum, "%s", str);
-
- free (str);
- }
-}
-/* ... likewise for error_at_line. */
-static void
-__attribute__ ((__format__ (__printf__, 5, 6), nonnull (3, 5), unused))
-record_error_at_line (int status, int errnum, const char *filename,
- unsigned int linenum, const char *format, ...)
-{
- char *str;
- va_list arg;
-
- recorded_error_count++;
-
- /* The existing behaviour is that even if you use --quiet, a fatal
- error is always printed and terminates the process. */
- if (!be_quiet || status != 0)
- {
- struct locale_state ls;
- int ret;
-
- va_start (arg, format);
- ls = push_locale ();
-
- ret = vasprintf (&str, format, arg);
- if (ret == -1)
- abort ();
-
- pop_locale (ls);
- va_end (arg);
-
- error_at_line (status, errnum, filename, linenum, "%s", str);
+struct locale_state push_locale (void);
+void pop_locale (struct locale_state ls);
- free (str);
- }
-}
#endif
diff --git a/localedata/Makefile b/localedata/Makefile
index 75f71bb..a5f3c92 100644
--- a/localedata/Makefile
+++ b/localedata/Makefile
@@ -200,17 +200,25 @@ install-locales: $(INSTALL-SUPPORTED-LOCALES)
install-locales-dir:
$(..)./scripts/mkinstalldirs $(inst_complocaledir)
+# The SHIFT_JIS and SHIFT_JISX0213 character maps are not ASCII compatible,
+# therefore we have to use --no-warnings=ascii to disable the ASCII check.
+# See localedata/gen-locale.sh for the same logic.
$(INSTALL-SUPPORTED-LOCALES): install-locales-dir
@locale=`echo $@ | sed -e 's/^install-//'`; \
charset=`echo $$locale | sed -e 's,.*/,,'`; \
locale=`echo $$locale | sed -e 's,/[^/]*,,'`; \
+ flags="--quiet -c"; \
+ if [ "$$charset" = 'SHIFT_JIS' ] \
+ || [ "$$charset" = 'SHIFT_JISX0213' ]; then \
+ flags="$$flags --no-warnings=ascii"; \
+ fi; \
echo -n `echo $$locale | sed 's/\([^.\@]*\).*/\1/'`; \
echo -n ".$$charset"; \
echo -n `echo $$locale | sed 's/\([^\@]*\)\(\@.*\)*/\2/'`; \
echo -n '...'; \
input=`echo $$locale | sed 's/\([^.]*\)[^@]*\(.*\)/\1\2/'`; \
- $(LOCALEDEF) --alias-file=../intl/locale.alias \
- -i locales/$$input -c -f charmaps/$$charset \
+ $(LOCALEDEF) $$flags --alias-file=../intl/locale.alias \
+ -i locales/$$input -f charmaps/$$charset \
$(addprefix --prefix=,$(install_root)) $$locale \
&& echo ' done'; \
diff --git a/localedata/gen-locale.sh b/localedata/gen-locale.sh
index b4ec68c..757a0e9 100644
--- a/localedata/gen-locale.sh
+++ b/localedata/gen-locale.sh
@@ -30,16 +30,12 @@ generate_locale ()
charmap=$1
input=$2
out=$3
+ flags=$4
ret=0
${localedef_before_env} ${run_program_env} I18NPATH=../localedata \
- ${localedef_after_env} --quiet -c -f $charmap -i $input \
+ ${localedef_after_env} $flags -f $charmap -i $input \
${common_objpfx}localedata/$out || ret=$?
- # All locales compile fine, except those with SHIFT_JIS charmap
- # and those fail with exit code 1 because SHIFT_JIS issues a
- # warning (it is not ASCII compatible).
- if [ $ret -eq 0 ] \
- || ( [ $ret -eq 1 ] \
- && [ "$charmap" = "SHIFT_JIS" ] ); then
+ if [ $ret -eq 0 ]; then
# The makefile checks the timestamp of the LC_CTYPE file,
# but localedef won't have touched it if it was able to
# hard-link it to an existing file.
@@ -57,6 +53,9 @@ charmap=`echo $locfile|sed 's|[^.]*[.]\(.*\)/LC_CTYPE|\1|'`
echo "Generating locale $locale.$charmap: this might take a while..."
+# Run quietly and force output.
+flags="--quiet -c"
+
# For SJIS the charmap is SHIFT_JIS. We just want the locale to have
# a slightly nicer name instead of using "*.SHIFT_SJIS", but that
# means we need a mapping here.
@@ -65,4 +64,13 @@ if [ "$charmap" = "SJIS" ]; then
charmap_real="SHIFT_JIS"
fi
-generate_locale $charmap_real $locale $locale.$charmap
+# In addition to this the SHIFT_JIS character maps are not ASCII
+# compatible so we must use `--no-warnings=ascii' to disable the
+# warning. See localedata/Makefile $(INSTALL-SUPPORTED-LOCALES)
+# for the same logic.
+if [ "$charmap_real" = 'SHIFT_JIS' ] \
+ || [ "$charmpa_real" = 'SHIFT_JISX0213' ]; then
+ flags="$flags --no-warnings=ascii"
+fi
+
+generate_locale $charmap_real $locale $locale.$charmap "$flags"
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=56fa555a834c1536bf8d58c1ac6097f18f0d92b6
commit 56fa555a834c1536bf8d58c1ac6097f18f0d92b6
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Fri Oct 13 22:44:44 2017 -0700
localedata: Locale and test name are the same.
The localedata collation test data is encoded in a particular
character set. We rename the test data to match the full locale
name with encoding, and adjust the Makefile and sort-test.sh
script. This allows us to have a future C.UTF-8 test that is
disambiguated from the built-in C locale.
Signed-off-by: Carlos O'Donell <carlos@redhat.com>
diff --git a/ChangeLog b/ChangeLog
index 070d07f..9ae9618 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,29 @@
2017-10-25 Carlos O'Donell <carlos@redhat.com>
+ * localedata/Makefile (test-input-data): Use full file name.
+ * localedata/da_DK.in: Rename to...
+ * localedata/da_DK.ISO-8859-1.in: ...this.
+ * localedata/de_DE.in: Rename to...
+ * localedata/de_DE.ISO-8859-1.in: ...this.
+ * localedata/en_US.in: Rename to...
+ * localedata/en_US.ISO-8859-1.in: ...this.
+ * localedata/fr_FR.in: Rename to...
+ * localedata/fr_FR.UTF-8.in: ... this.
+ * localedata/hr_HR.in: Rename to...
+ * localedata/hr_HR.ISO-8859-2.in: ...this.
+ * localedata/hu_HU.in: Rename to...
+ * localedata/hu_HU.UTF-8.in: ...this.
+ * localedata/si_LK.in: Rename to...
+ * localedata/si_LK.UTF-8.in: ...this.
+ * localedata/sv_SE.in: Rename to...
+ * localedata/sv_SE.ISO-8859-1.in: ...this.
+ * localedata/tr_TR.in: Rename to...
+ * localedata/tr_TR.UTF-8.in: ...this.
+ * localedata/uk_UA.in: Rename to...
+ * localedata/uk_UA.UTF-8.in: ...this.
+ * localedata/sort-test.sh: Test file is locale name with the
+ suffix.
+
* localedata/unicode-gen/Makefile (check_i18n): Rename to
check_i18n_ctype. Depend on i18n_ctype-report. Check i18n_ctype-report
file.
@@ -995,14 +1019,10 @@
* locale/programs/linereader.c (get_string): Don't warn on
non-symbolic character.
-2017-10-13 Carlos O'Donell <carlos@redhat.com>
-
[BZ #22294]
* locale/programs/ld-monetary.c (monetary_finish): Allow ""
int_curr_symbol.
-2017-10-13 Carlos O'Donell <carlos@redhat.com>
-
[BZ #22292]
* locale/programs/record-status.h: New file
* locale/programs/locale.c: Add comment.
@@ -1059,8 +1079,6 @@
* localedata/tst-locale.sh: Likewise.
* localedata/gen-locale.sh: Expect failures from SHIFT_JIS.
-2017-10-12 Carlos O'Donell <carlos@redhat.com>
-
* localedata/unicode-gen/Makefile (GENERATED): Use i18n_ctype.
(REPORTS): Likewise.
(check): Likewise.
diff --git a/localedata/Makefile b/localedata/Makefile
index 84dbc5a..75f71bb 100644
--- a/localedata/Makefile
+++ b/localedata/Makefile
@@ -38,7 +38,7 @@ test-srcs := collate-test xfrm-test tst-fmon tst-rpmatch tst-trans \
test-input := de_DE.ISO-8859-1 en_US.ISO-8859-1 da_DK.ISO-8859-1 \
hr_HR.ISO-8859-2 sv_SE.ISO-8859-1 tr_TR.UTF-8 fr_FR.UTF-8 \
si_LK.UTF-8 uk_UA.UTF-8 hu_HU.UTF-8
-test-input-data = $(addsuffix .in, $(basename $(test-input)))
+test-input-data = $(addsuffix .in, $(test-input))
test-output := $(foreach s, .out .xout, \
$(addsuffix $s, $(basename $(test-input))))
ld-test-names := test1 test2 test3 test4 test5 test6 test7
diff --git a/localedata/da_DK.in b/localedata/da_DK.ISO-8859-1.in
similarity index 100%
rename from localedata/da_DK.in
rename to localedata/da_DK.ISO-8859-1.in
diff --git a/localedata/de_DE.in b/localedata/de_DE.ISO-8859-1.in
similarity index 100%
rename from localedata/de_DE.in
rename to localedata/de_DE.ISO-8859-1.in
diff --git a/localedata/en_US.in b/localedata/en_US.ISO-8859-1.in
similarity index 100%
rename from localedata/en_US.in
rename to localedata/en_US.ISO-8859-1.in
diff --git a/localedata/fr_FR.in b/localedata/fr_FR.UTF-8.in
similarity index 100%
rename from localedata/fr_FR.in
rename to localedata/fr_FR.UTF-8.in
diff --git a/localedata/hr_HR.in b/localedata/hr_HR.ISO-8859-2.in
similarity index 100%
rename from localedata/hr_HR.in
rename to localedata/hr_HR.ISO-8859-2.in
diff --git a/localedata/hu_HU.in b/localedata/hu_HU.UTF-8.in
similarity index 100%
rename from localedata/hu_HU.in
rename to localedata/hu_HU.UTF-8.in
diff --git a/localedata/si_LK.in b/localedata/si_LK.UTF-8.in
similarity index 100%
rename from localedata/si_LK.in
rename to localedata/si_LK.UTF-8.in
diff --git a/localedata/sort-test.sh b/localedata/sort-test.sh
index 46ed5f0..17fb262 100644
--- a/localedata/sort-test.sh
+++ b/localedata/sort-test.sh
@@ -32,7 +32,8 @@ here=`pwd`
status=0
for l in $lang; do
here=0
- cns=`echo $l | sed 's/\(.*\)[.][^.]*/\1/'`
+ # Locale name matches test filename (without in).
+ cns=$l
${test_program_prefix_before_env} \
${run_program_env} \
LC_ALL=$l ${test_program_prefix_after_env} \
diff --git a/localedata/sv_SE.in b/localedata/sv_SE.ISO-8859-1.in
similarity index 100%
rename from localedata/sv_SE.in
rename to localedata/sv_SE.ISO-8859-1.in
diff --git a/localedata/tr_TR.in b/localedata/tr_TR.UTF-8.in
similarity index 100%
rename from localedata/tr_TR.in
rename to localedata/tr_TR.UTF-8.in
diff --git a/localedata/uk_UA.in b/localedata/uk_UA.UTF-8.in
similarity index 100%
rename from localedata/uk_UA.in
rename to localedata/uk_UA.UTF-8.in
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 59 +++++-
iconv/Makefile | 3 +-
iconv/iconv_prog.c | 3 -
iconv/iconv_prog.h | 6 +-
locale/Makefile | 3 +-
locale/programs/charmap.c | 12 +-
locale/programs/ld-monetary.c | 9 +-
locale/programs/locale.c | 8 +-
locale/programs/localedef.c | 67 +++++-
locale/programs/localedef.h | 1 -
.../programs/{record-status.h => record-status.c} | 58 +++---
locale/programs/record-status.h | 220 ++------------------
localedata/Makefile | 14 +-
localedata/{da_DK.in => da_DK.ISO-8859-1.in} | 0
localedata/{de_DE.in => de_DE.ISO-8859-1.in} | 0
localedata/{en_US.in => en_US.ISO-8859-1.in} | 0
localedata/{fr_FR.in => fr_FR.UTF-8.in} | 0
localedata/gen-locale.sh | 24 ++-
localedata/{hr_HR.in => hr_HR.ISO-8859-2.in} | 0
localedata/{hu_HU.in => hu_HU.UTF-8.in} | 0
localedata/{si_LK.in => si_LK.UTF-8.in} | 0
localedata/sort-test.sh | 3 +-
localedata/{sv_SE.in => sv_SE.ISO-8859-1.in} | 0
localedata/{tr_TR.in => tr_TR.UTF-8.in} | 0
localedata/{uk_UA.in => uk_UA.UTF-8.in} | 0
25 files changed, 212 insertions(+), 278 deletions(-)
copy locale/programs/{record-status.h => record-status.c} (84%)
rename localedata/{da_DK.in => da_DK.ISO-8859-1.in} (100%)
rename localedata/{de_DE.in => de_DE.ISO-8859-1.in} (100%)
rename localedata/{en_US.in => en_US.ISO-8859-1.in} (100%)
rename localedata/{fr_FR.in => fr_FR.UTF-8.in} (100%)
rename localedata/{hr_HR.in => hr_HR.ISO-8859-2.in} (100%)
rename localedata/{hu_HU.in => hu_HU.UTF-8.in} (100%)
rename localedata/{si_LK.in => si_LK.UTF-8.in} (100%)
rename localedata/{sv_SE.in => sv_SE.ISO-8859-1.in} (100%)
rename localedata/{tr_TR.in => tr_TR.UTF-8.in} (100%)
rename localedata/{uk_UA.in => uk_UA.UTF-8.in} (100%)
hooks/post-receive
--
GNU C Library master sources