mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-08 14:20:07 +00:00
localedata: Do not generate output if warnings were present.
With LC_MONETARY parsing fixed we can now generate locales without forcing output with '-c'. Removing '-c' from localedef invocation is the equivalent of using -Werror for localedef. The glibc locale sources should always be clean and free from warnings. We remove '-c' from both test locale generation and the targets used for installing locales e.g. install-locale-archive, and install-locale-files. Tested on x86_64 and i686 without regressions. Tested with install-locale-archive target. Tested with install-locale-files target. Reviewed-by: DJ Delorie <dj@redhat.com>
This commit is contained in:
parent
2ab8b74567
commit
1c7a34567d
@ -468,11 +468,11 @@ define build-one-locale
|
||||
endef
|
||||
|
||||
$(INSTALL-SUPPORTED-LOCALE-ARCHIVE): install-locales-dir
|
||||
@flags="-c"; \
|
||||
@flags=""; \
|
||||
$(build-one-locale)
|
||||
|
||||
$(INSTALL-SUPPORTED-LOCALE-FILES): install-locales-dir
|
||||
@flags="-c --no-archive --no-hard-links"; \
|
||||
@flags="--no-archive --no-hard-links"; \
|
||||
$(build-one-locale)
|
||||
|
||||
tst-setlocale-ENV = LC_ALL=ja_JP.EUC-JP
|
||||
|
@ -54,8 +54,14 @@ modifier=`echo $locfile|sed 's|[^.]*[.]\([^@ ]*\)\(@[^ ]*\)\?/LC_CTYPE|\2|'`
|
||||
|
||||
echo "Generating locale $locale.$charmap: this might take a while..."
|
||||
|
||||
# Run quietly and force output.
|
||||
flags="--quiet -c"
|
||||
# Do not force output with '-c', all locales should compile without
|
||||
# warning or errors. There is likewise no need to run quietly with
|
||||
# '--quiet' since all locales should compile without additional
|
||||
# diagnostics. If there are messages printed then we want to see
|
||||
# them, fix them, and the associated error or warning. During
|
||||
# development it may be beneficialy to put '--quiet -c' here to allow
|
||||
# you to develop in-progress locales.
|
||||
flags=""
|
||||
|
||||
# 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
|
||||
|
Loading…
Reference in New Issue
Block a user