[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request 129691: digital-clock: Update iso-3166 codes in timezones
From: Jiri Bohac <jiri () boha ! cz>
Date: 2016-12-27 16:30:52
Message-ID: 20161227163052.23879.57133 () mimi ! kde ! org
[Download RAW message or body]
--===============3788084200709167498==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
> On Dec. 27, 2016, 3:54 p.m., Sebastian Kügler wrote:
> > applets/digital-clock/plugin/timezonesi18n.cpp, line 682
> > <https://git.reviewboard.kde.org/r/129691/diff/1/?file=487931#file487931line682>
> >
> > This should stay Cape Verde, the English name is used here, not the Portuguese \
> > (?) one.
Frankly, I don't understand what all the discussion here is about.
I noticed the list of countries, obviously taken from iso-codes some day in the past, \
was out of date. The comment above the hash table clearly says "Make sure the country \
names match their versions in iso-codes". So I went and fixed everything to the \
current version of the ISO standard.
Suddenly, people start disputing the changes with reasoning like:
"This should stay Cape Verde, the English name is used here, not the Portuguese"
"Czech Republic" is correct as well, since it's the official name"
"Congo" is IMHO way too ambiguous
I am no expert on naming countries. There are whole commities doing that at UN and \
ISO. They decided that "Czechia", "Congo" and "Cabo Verde" are the correct short \
names and updated their databases/standards.
I would have loved [not] to see such a discussion when populating this hash table \
from scratch :))
I find it totally unsystematic to maitain a few records out of sync with iso-codes. \
Either by using an old version (Cabo Verde) or not using a newly standardized short \
version. Why would anyone think the snapshot of iso_3116 from ~1 year ago was good \
and suddenly new ammendment to the standard are controversial?
If someone, who is a better expert on international politics and territory naming \
than the UN, feels they want to use any subset of my changes, feel free to do that. I \
totally don't understand any reasoning behind that, though.
- Jiri
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129691/#review101595
-----------------------------------------------------------
On Dec. 20, 2016, 9:57 p.m., Jiri Bohac wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129691/
> -----------------------------------------------------------
>
> (Updated Dec. 20, 2016, 9:57 p.m.)
>
>
> Review request for Plasma and Albert Astals Cid.
>
>
> Repository: plasma-workspace
>
>
> Description
> -------
>
> Update the list of country names to the current names in iso-codes
> (as output by isoquery -c).
>
> The motivation is updating the name of Czechia as a followup to
> https://git.reviewboard.kde.org/r/129644/
> As suggested by Albert Astals Cid, fixing the name in i10n is not enough:
> country names are hardcoded in many places and they need to be fixed separately.
>
>
> Diffs
> -----
>
> applets/digital-clock/plugin/timezonesi18n.cpp 10650f8
>
> Diff: https://git.reviewboard.kde.org/r/129691/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jiri Bohac
>
>
--===============3788084200709167498==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 8bit
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 \
solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;"> \
<tr> <td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129691/">https://git.reviewboard.kde.org/r/129691/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On December 27th, 2016, 3:54 p.m. UTC, \
<b>Sebastian Kügler</b> wrote:</p> <blockquote style="margin-left: 1em; \
border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="https://git.reviewboard.kde.org/r/129691/diff/1/?file=487931#file487931line682" \
style="color: black; font-weight: bold; text-decoration: \
underline;">applets/digital-clock/plugin/timezonesi18n.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2">680</font></th> <td bgcolor="#fdfebc" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="n">ENTRY_ISO_3166</span><span class="p">(</span><span \
class="n">CapeVerde</span><span class="p">,</span> <span class="s">"Ca<span \
class="hl">pe</span> Verde"</span><span class="p">),</span></pre></td> <th \
bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">682</font></th> <td bgcolor="#fdfebc" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="n">ENTRY_ISO_3166</span><span class="p">(</span><span \
class="n">CapeVerde</span><span class="p">,</span> <span class="s">"Ca<span \
class="hl">bo</span> Verde"</span><span class="p">),</span></pre></td> </tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This \
should stay Cape Verde, the English name is used here, not the Portuguese (?) \
one.</p></pre> </blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">Frankly, I don't understand what all the discussion here is about. I \
noticed the list of countries, obviously taken from iso-codes some day in the past, \
was out of date. The comment above the hash table clearly says "Make sure the country \
names match their versions in iso-codes". So I went and fixed everything to the \
current version of the ISO standard. </p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">Suddenly, people start \
disputing the changes with reasoning like: "This should stay Cape Verde, the English \
name is used here, not the Portuguese" "Czech Republic" is correct as well, since \
it's the official name" "Congo" is IMHO way too ambiguous</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">I am no expert on naming countries. There are whole \
commities doing that at UN and ISO. They decided that "Czechia", "Congo" and "Cabo \
Verde" are the correct short names and updated their databases/standards.</p> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">I would have loved [not] to see such a discussion when populating this hash \
table from scratch :))</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">I find it totally unsystematic to \
maitain a few records out of sync with iso-codes. Either by using an old version \
(Cabo Verde) or not using a newly standardized short version. Why would anyone think \
the snapshot of iso_3116 from ~1 year ago was good and suddenly new ammendment to the \
standard are controversial?</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">If someone, who is a better expert on \
international politics and territory naming than the UN, feels they want to use any \
subset of my changes, feel free to do that. I totally don't understand any reasoning \
behind that, though.</p></pre> <br />
<p>- Jiri</p>
<br />
<p>On December 20th, 2016, 9:57 p.m. UTC, Jiri Bohac wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: \
1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; \
-webkit-border-radius: 6px;"> <tr>
<td>
<div>Review request for Plasma and Albert Astals Cid.</div>
<div>By Jiri Bohac.</div>
<p style="color: grey;"><i>Updated Dec. 20, 2016, 9:57 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-workspace
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Update the list of country names to the current names in iso-codes (as \
output by isoquery -c).
The motivation is updating the name of Czechia as a followup to
https://git.reviewboard.kde.org/r/129644/
As suggested by Albert Astals Cid, fixing the name in i10n is not enough:
country names are hardcoded in many places and they need to be fixed \
separately.</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>applets/digital-clock/plugin/timezonesi18n.cpp <span style="color: \
grey">(10650f8)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129691/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
--===============3788084200709167498==--
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic