Modify

Opened 9 months ago

Last modified 9 months ago

#12997 assigned Patch - Bug Fix

[PATCH] Country name correction

Reported by: cypressyew@… Owned by: knight
Priority: minor Milestone: unknown
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Country name correction for future localization tasks (zh_TW).

https://github.com/MythTV/mythtv/pull/128

Reference (if anyone is interested): https://en.wikipedia.org/wiki/Taiwan

Attachments (0)

Change History (3)

comment:1 Changed 9 months ago by knight

  • Owner set to knight
  • Status changed from new to assigned

comment:2 Changed 9 months ago by dekarl

Thank you for looking into this.

I think it is ok to deviate from ISO 3166 and follow Musicbrainz' variant of the ISO 3166 country list. (They generally set good precedent, like adding plain north/south to the korean entries, so layman know which one is which.) https://en.wikipedia.org/wiki/ISO_3166-1#Naming_and_disputes has a note on the country name in the standard.

locales.py mentions that the list is automatically generated (But how? And from what source?). Please keep our other locale list in sync until we find out how to do so automatically. See https://github.com/MythTV/mythtv/blob/master/mythtv/libs/libmythbase/iso3166.cpp#L312

comment:3 Changed 9 months ago by cypressyew@…

Hello, thanks for the reply, I have iso3166.cpp updated as well.

I searched again in the repo, and it looks like the locale.py was generated by running ../scripts/populate_locale.py script in the tmdb3 directory. However it looks like it's broken now, probably there's something to do with the format change in the target ISO 3166 webpage. As you mentioned that we can use Musicbrainz' variant [1], maybe we can change the populate_locale.py to do this in the future.

Thanks

[1] https://wiki.musicbrainz.org/Release_Country

Add Comment

Modify Ticket

Action
as assigned The owner will remain knight.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.