diff mbox

dsb_DE locale: Fix syntax error and add tests (bug 23208).

Message ID 746200491.50429.1531346344599@poczta.nazwa.pl
State Committed
Headers

Commit Message

Rafal Luzynski July 11, 2018, 9:59 p.m. UTC 
  Fixed syntax error in the collation rules of Lower Sorbian language.
Collation test added in order to test the bugs like this early.

Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>

	[BZ #23208]
	* localedata/Makefile (test-input): Add dsb_DE.UTF-8.
	(LOCALES): Likewise.
	* localedata/dsb_DE.UTF-8.in: New file.
	* localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
---
 localedata/Makefile        |  2 ++
 localedata/dsb_DE.UTF-8.in | 73 ++++++++++++++++++++++++++++++++++++++++++++++
 localedata/locales/dsb_DE  |  4 +--
 3 files changed, 77 insertions(+), 2 deletions(-)
 create mode 100644 localedata/dsb_DE.UTF-8.in

Comments

Tulio Magno Quites Machado Filho July 13, 2018, 3:02 p.m. UTC | #1 
Rafal Luzynski <digitalfreak@lingonborough.com> writes:

> Fixed syntax error in the collation rules of Lower Sorbian language.
> Collation test added in order to test the bugs like this early.
>
> Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>
> 	[BZ #23208]
> 	* localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> 	(LOCALES): Likewise.
> 	* localedata/dsb_DE.UTF-8.in: New file.
> 	* localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.

I confirmed this patch does fix the issue.

Thanks!
Rafal Luzynski July 13, 2018, 7:47 p.m. UTC | #2 
13.07.2018 17:02 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> wrote:
>
> Rafal Luzynski <digitalfreak@lingonborough.com> writes:
>
> > Fixed syntax error in the collation rules of Lower Sorbian language.
> > Collation test added in order to test the bugs like this early.
> >
> > Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
> >
> > [BZ #23208]
> > * localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> > (LOCALES): Likewise.
> > * localedata/dsb_DE.UTF-8.in: New file.
> > * localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
>
> I confirmed this patch does fix the issue.
>
> Thanks!

Since the patch fixes an actual bug and changes of the locale data are
usually not harmful for the rest of the project I assume this review
is sufficient and I'm going to push this patch now.

Regards,

Rafal
Rafal Luzynski July 13, 2018, 9:09 p.m. UTC | #3 
11.07.2018 23:59 Rafal Luzynski <digitalfreak@lingonborough.com> wrote:
>
> Fixed syntax error in the collation rules of Lower Sorbian language.
> Collation test added in order to test the bugs like this early.
>
> Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>
> [BZ #23208]
> * localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> (LOCALES): Likewise.
> * localedata/dsb_DE.UTF-8.in: New file.
> * localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
> ---

This patch has been pushed to master.

Regards,

Rafal
diff mbox

Patch

diff --git a/localedata/Makefile b/localedata/Makefile
index 2e6e0dc..5ac4f74 100644
--- a/localedata/Makefile
+++ b/localedata/Makefile
@@ -55,6 +55,7 @@  test-input := \
 	cy_GB.UTF-8 \
 	da_DK.ISO-8859-1 \
 	de_DE.ISO-8859-1 \
+	dsb_DE.UTF-8 \
 	dz_BT.UTF-8 \
 	en_US.ISO-8859-1 \
 	eo.UTF-8 \
@@ -200,6 +201,7 @@  LOCALES := \
 	da_DK.ISO-8859-1 \
 	de_DE.ISO-8859-1 \
 	de_DE.UTF-8 \
+	dsb_DE.UTF-8 \
 	dz_BT.UTF-8 \
 	en_GB.UTF-8 \
 	en_US.ANSI_X3.4-1968 \
diff --git a/localedata/dsb_DE.UTF-8.in b/localedata/dsb_DE.UTF-8.in
new file mode 100644
index 0000000..adc5713
--- /dev/null
+++ b/localedata/dsb_DE.UTF-8.in
@@ -0,0 +1,73 @@ 
+a
+A
+b
+B
+c
+C
+č
+Č
+ć
+d
+D
+dź
+Dź
+e
+E
+Ä›
+f
+F
+g
+G
+h
+H
+ch
+Ch
+i
+I
+j
+J
+k
+K
+Å‚
+Ł
+l
+L
+m
+M
+n
+N
+Å„
+o
+O
+ó
+p
+P
+q
+Q
+r
+R
+Å•
+s
+S
+Å¡
+Å 
+Å›
+Åš
+t
+T
+u
+U
+v
+V
+w
+W
+x
+X
+y
+Y
+z
+Z
+ž
+Ž
+ź
+Ź
diff --git a/localedata/locales/dsb_DE b/localedata/locales/dsb_DE
index 419d0b3..7f51b82 100644
--- a/localedata/locales/dsb_DE
+++ b/localedata/locales/dsb_DE
@@ -77,8 +77,8 @@  collating-symbol <d-z-acute-digraph>
 collating-symbol <e-caron>
 collating-symbol <c-h-digraph>
 collating-symbol <l-stroke>
-collating symbol <n-acute>
-collating symbol <o-acute>
+collating-symbol <n-acute>
+collating-symbol <o-acute>
 collating-symbol <r-acute>
 collating-symbol <s-caron>
 collating-symbol <s-acute>