1906998 - genUnicodePropertyData.pl refers wrong VerticalOrientation list

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Takanori MATSUURA]

intl/unicharutil/tools/genUnicodePropertyData.pl reads VerticalOrientation data from https://www.unicode.org/Public/vertical/revision-17/VerticalOrientation-17.txt.

According to the recent UTR#50, we should read https://www.unicode.org/Public/UCD/latest/ucd/VerticalOrientation.txt (or https://www.unicode.org/Public/UNIDATA/VerticalOrientation.txt) instead.

Comment 1

[Takanori MATSUURA]

Or replace it with the function implemented in harfbuzz.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:m_kato, could you have a look please?

For more information, please visit BugBot documentation.

Comment 3

[Makoto Kato [:m_kato]]

ICU4X doesn't support this properties yet. I will add it for next release.

Comment 4

[Makoto Kato [:m_kato]]

Attachment: Bug 1906998 - Part 1. Use ICU4C to get vertical orientation properties. r=jfkthame

Our ICU's system requirement is 73.1 or higher, so we can use ICU API to
get vertical orientation properties.

Comment 5

[Makoto Kato [:m_kato]]

Attachment: Bug 1906998 - Part 2. Re-build unicode data without vertical orientation. r=jfkthame

The vertical orientation data is unnecessary now, so let's remove it.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa72f865d20c
Part 1. Use ICU4C to get vertical orientation properties. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/e5a497706356
Part 2. Re-build unicode data without vertical orientation. r=jfkthame

Comment 7

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/fa72f865d20c
https://hg.mozilla.org/mozilla-central/rev/e5a497706356