mirror of
https://sourceware.org/git/glibc.git
synced 2024-12-04 19:00:09 +00:00
intl/tst-gettext: fix failure with newest msgfmt
Since upstream gettext commit d13f165b83 (msgfmt: Remove
POT-Creation-Date field from the header in the output.), msgfmt does not
copy the POT-Creation-Date field in the header entry from the po file to
the mo file anymore. This breaks the assumption that we can test gettext
by comparing each message in the po files with the corresponding string
return by gettext. This makes the intl/tst-gettext to fail.
While it would have been possible to modify the po2test.awk script to
also strip the line POT-Creation-Date field when creating the msgs.h
file, it would not work with both the old and new msgfmt.
Instead create a tst-gettext-de.po file from de.po by removing the
POT-Creation-Date line. Another alternative would be to use a static
tst-gettext-de.po file, but I guess the reason for using de.po is to
also catch issues caused by newly added strings.
As tst-catgets also uses msg.h, it should also be updated. Instead of
using the new tst-gettext-de.po file, the patch modifies xopen-msg.awk
to avoid creating a second catgets->intl dependency.
Changelog:
[BZ #21508]
* catgets/xopen-msg.awk: Ignore POT-Creation-Date line.
* intl/Makefile ($(objpfx)tst-gettext-de.po): Generate
intl/tst-gettext-de.po from po/de.po by removing the
POT-Creation-Date line.
($(objpfx)msgs.h): Depend on $(objpfx)tst-gettext-de.po instead of
../po/de.po.
* intl/tst-gettext.sh: Use ${objpfx}tst-gettext-de.po instead of
../po/de.po.
(cherry picked from commit 56456a2aad
)
This commit is contained in:
parent
dc7f51bda9
commit
4b64a4245c
12
ChangeLog
12
ChangeLog
@ -1,3 +1,15 @@
|
||||
2018-02-18 Aurelien Jarno <aurelien@aurel32.net>
|
||||
|
||||
[BZ #21508]
|
||||
* catgets/xopen-msg.awk: Ignore POT-Creation-Date line.
|
||||
* intl/Makefile ($(objpfx)tst-gettext-de.po): Generate
|
||||
intl/tst-gettext-de.po from po/de.po by removing the
|
||||
POT-Creation-Date line.
|
||||
($(objpfx)msgs.h): Depend on $(objpfx)tst-gettext-de.po instead of
|
||||
../po/de.po.
|
||||
* intl/tst-gettext.sh: Use ${objpfx}tst-gettext-de.po instead of
|
||||
../po/de.po.
|
||||
|
||||
2018-07-06 Szabolcs Nagy <szabolcs.nagy@arm.com>
|
||||
|
||||
* sysdeps/unix/sysv/linux/aarch64/dl-procinfo.h (HWCAP_IMPORTANT): Add
|
||||
|
1
NEWS
1
NEWS
@ -488,6 +488,7 @@ The following bugs are resolved with this release:
|
||||
[21326] libc: C99 functions are not declared for C++11 and later if
|
||||
_GNU_SOURCE is not predefined by g++
|
||||
[21457] libc: sys/ucontext.h namespace
|
||||
[21508] locale: intl/tst-gettext failure with latest msgfmt
|
||||
[21530] libc: tmpfile() should be implemented using O_TMPFILE
|
||||
[21660] math: GCC fails to compile a formula with tgmath.h
|
||||
[21672] nptl: sys-libs/glibc on ia64 crashes on thread exit: signal
|
||||
|
@ -58,6 +58,11 @@ $1 ~ "msg(id|str)" {
|
||||
next
|
||||
}
|
||||
|
||||
/^"POT-Creation-Date: [0-9-]+ [0-9:+-]+\\n"/ {
|
||||
# Ignore POT-Creation-Date to match what is done in intl/Makefile.
|
||||
next
|
||||
}
|
||||
|
||||
/^".*"/ {
|
||||
# Append to current message
|
||||
sub(/^"/, "", $0)
|
||||
|
@ -120,7 +120,11 @@ $(objpfx)tst-gettext6.out: $(gen-locales)
|
||||
$(objpfx)tst-translit.out: $(gen-locales)
|
||||
endif
|
||||
|
||||
$(objpfx)msgs.h: po2test.awk ../po/de.po
|
||||
$(objpfx)tst-gettext-de.po: ../po/de.po
|
||||
$(make-target-directory)
|
||||
LC_ALL=C $(AWK) '!/^"POT-Creation-Date: [0-9-]+ [0-9:+-]+\\n"$$/' $^ > $@
|
||||
|
||||
$(objpfx)msgs.h: po2test.awk $(objpfx)tst-gettext-de.po
|
||||
$(make-target-directory)
|
||||
LC_ALL=C $(AWK) -f $^ > $@
|
||||
|
||||
|
@ -42,9 +42,9 @@ mkdir -p ${objpfx}domaindir/existing-locale/LC_MESSAGES
|
||||
mkdir -p ${objpfx}domaindir/existing-locale/LC_TIME
|
||||
# Populate them.
|
||||
msgfmt -o ${objpfx}domaindir/existing-locale/LC_MESSAGES/existing-domain.mo \
|
||||
-f ../po/de.po
|
||||
-f ${objpfx}tst-gettext-de.po
|
||||
msgfmt -o ${objpfx}domaindir/existing-locale/LC_TIME/existing-time-domain.mo \
|
||||
-f ../po/de.po
|
||||
-f ${objpfx}tst-gettext-de.po
|
||||
|
||||
# Now run the test.
|
||||
${test_program_prefix_before_env} \
|
||||
|
Loading…
Reference in New Issue
Block a user