[1/1] Fix LC_TELEPHONE/LC_NAME for az_AZ

Message ID 1504840130-42253-1-git-send-email-akhilesh.k@samsung.com
State New
Headers show
Series
  • [1/1] Fix LC_TELEPHONE/LC_NAME for az_AZ
Related show

Commit Message

AKHILESH KUMAR Sept. 8, 2017, 3:08 a.m.
Fix LC_TELEPHONE/LC_NAME for az_AZ

[BZ #22112]
	*locales/az_AZ(LC_TELEPHONE) : Fix  int_select
	*locales/az_AZ(LC_TELEPHONE) : add  tel_int_fmt
	*locales/az_AZ(LC_NAME): copy "az_IR"
---
 localedata/locales/az_AZ |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)

Comments

Mike FABIAN Sept. 8, 2017, 9:19 a.m. | #1
Akhilesh Kumar <akhilesh.k@samsung.com> wrote:

> Fix LC_TELEPHONE/LC_NAME for az_AZ
>
> [BZ #22112]
> 	*locales/az_AZ(LC_TELEPHONE) : Fix  int_select
> 	*locales/az_AZ(LC_TELEPHONE) : add  tel_int_fmt
> 	*locales/az_AZ(LC_NAME): copy "az_IR"
> ---
>  localedata/locales/az_AZ |   10 ++++------
>  1 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/localedata/locales/az_AZ b/localedata/locales/az_AZ
> index 28de66d..7a17bb5 100644
> --- a/localedata/locales/az_AZ
> +++ b/localedata/locales/az_AZ
> @@ -311,13 +311,11 @@ lang_lib    "<U0061><U007A><U0065>"
>  END LC_ADDRESS
>  
>  LC_TELEPHONE
> -% FIXME
> -tel_int_fmt "???"
> -int_prefix "<U0039><U0039><U0034>"
> -int_select "<U0038><U007E><U0031><U0030>"
> +tel_int_fmt "+%c %l"

Shouldn’t this be "+%c %a %l"?

https://www.howtocallabroad.com/results.php?callfrom=azerbaijan&callto=azerbaijan

has an additional 0 as a prefix before the area codes. And

https://www.howtocallabroad.com/results.php?callfrom=germany&callto=azerbaijan

omits that 0.

From man 5 locale:

     tel_int_fmt
          followed by  a string  that contains  field descriptors
          that  identify the  format used  to dial  international
          numbers.    The   following   field   descriptors   are
          recognized:

          %a  Area code without nationwide  prefix (the prefix is
              often "00").

          %A  Area code including nationwide prefix.

          %l  Local number (within area code).

So this makes me think that "+%c %a %l" is correct.

By the way, in it_IT we have:

    tel_int_fmt "+%c %a %l"

but when calling to Italy for example from Germany, the prefix 0 before
the area code is *not* omitted:

https://www.howtocallabroad.com/results.php?callfrom=germany&callto=italy

So I wonder whether it_IT should have

    tel_int_fmt "+%c %A %l"

instead ...

> +int_prefix "994"
> +int_select "00"
>  END LC_TELEPHONE
>  
>  LC_NAME
> -% FIXME
> -name_fmt "???"
> +copy "az_IR"
>  END LC_NAME

az_AZ seems to use Latin script:

    % Azeri Language Locale for Azerbaijan (latin)

If you look at LC_TIME, you see that Latin script is used for the day
and month names.

But az_IR uses Arabic script. And LC_NAME in az_IR looks like:

    LC_NAME
    name_gen ""
    % Xanim:
    name_miss "خانیم"
    name_ms "خانیم"
    name_mrs "خانیم"
    % Agha
    name_mr "آغا"
    name_fmt "%d%t%s%t%g%t%m%t%f"
    END LC_NAME

So I don’t think this “copy "az_IR"” in LC_NAME is correct.
AKHILESH KUMAR Sept. 14, 2017, 3:29 a.m. | #2
>>  
>>  LC_TELEPHONE
>> -% FIXME
>> -tel_int_fmt "???"
>> -int_prefix "<U0039><U0039><U0034>"
>> -int_select "<U0038><U007E><U0031><U0030>"
>> +tel_int_fmt "+%c %l"
> 
>Shouldn’t this be "+%c %a %l"?


Agree please find updated patch 
https://sourceware.org/ml/libc-alpha/2017-09/msg00584.html

>https://www.howtocallabroad.com/results.php?callfrom=azerbaijan&callto=azerbaijan

> 

>has an additional 0 as a prefix before the area codes. And

> 

>https://www.howtocallabroad.com/results.php?callfrom=germany&callto=azerbaijan

> 

>omits that 0.


 
>az_AZ seems to use Latin script:

> 

>    % Azeri Language Locale for Azerbaijan (latin)

> 

>If you look at LC_TIME, you see that Latin script is used for the day

>and month names.

> 

>But az_IR uses Arabic script. And LC_NAME in az_IR looks like:

> 

>    LC_NAME

>    name_gen ""

>    % Xanim:

>    name_miss "خانیم"

>    name_ms "خانیم"

>    name_mrs "خانیم"

>    % Agha

>    name_mr "آغا"

>    name_fmt "%d%t%s%t%g%t%m%t%f"

>    END LC_NAME

> 

>So I don’t think this “copy "az_IR"” in LC_NAME is correct.

Currently Agree

Patch

diff --git a/localedata/locales/az_AZ b/localedata/locales/az_AZ
index 28de66d..7a17bb5 100644
--- a/localedata/locales/az_AZ
+++ b/localedata/locales/az_AZ
@@ -311,13 +311,11 @@  lang_lib    "<U0061><U007A><U0065>"
 END LC_ADDRESS
 
 LC_TELEPHONE
-% FIXME
-tel_int_fmt "???"
-int_prefix "<U0039><U0039><U0034>"
-int_select "<U0038><U007E><U0031><U0030>"
+tel_int_fmt "+%c %l"
+int_prefix "994"
+int_select "00"
 END LC_TELEPHONE
 
 LC_NAME
-% FIXME
-name_fmt "???"
+copy "az_IR"
 END LC_NAME