[Bug localedata/22646] New: redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[Bug localedata/22646] New: redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[akhilesh.k at samsung dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

            Bug ID: 22646
           Summary: redundant data (LC_TIME) for es_CL, es_CU, es_EC and
                    es_BO
           Product: glibc
           Version: 2.26
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: akhilesh.k at samsung dot com
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

Hello 
Seems in locale there is Redundant data (LC_TIME) for es_CL, es_CU, es_EC and
es_BO. 

Thanks
Akhilesh

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[akhilesh.k at samsung dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

--- Comment #1 from Akhilesh Kumar <akhilesh.k at samsung dot com> ---
Created attachment 10700
  --> https://sourceware.org/bugzilla/attachment.cgi?id=10700&action=edit
Removed redundant data (LC_TIME) for es_CL, es_CU, es_EC [BZ #22646]

Removed redundant data (LC_TIME) for es_CL, es_CU, es_EC [BZ #22646]
Bolivia is taken as reference because its official language is Spanish

[BZ #22646]
* locales/es_CL  (LC_TIME): copy "es_BO".
* locales/es_CU  (LC_TIME): copy "es_BO"..
* locales/es_EC  (LC_TIME): copy "es_BO".

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |digitalfreak@lingonborough.
                   |                            |com

--- Comment #2 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Probably the same in more es_* locales except es_PE and es_UY, as far as I
know.

(In reply to Akhilesh Kumar from comment #1)
> Created attachment 10700 [details]
> Removed redundant data (LC_TIME) for es_CL, es_CU, es_EC [BZ #22646]
> 
> Removed redundant data (LC_TIME) for es_CL, es_CU, es_EC [BZ #22646]
> Bolivia is taken as reference because its official language is Spanish

That's really not a sufficient reason because Spanish is an official language
in 20 more countries. Bolivia is not a special place in this group. I think
that it's better to use copy "es_ES" because Spain is an original location of
Spanish.

> [BZ #22646]
> * locales/es_CL  (LC_TIME): copy "es_BO".
> * locales/es_CU  (LC_TIME): copy "es_BO"..
> * locales/es_EC  (LC_TIME): copy "es_BO".

... therefore copy "es_ES" everywhere (where applicable, of course) including
es_BO.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[gunnarhj at ubuntu dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Gunnar Hjalmarsson <gunnarhj at ubuntu dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gunnarhj at ubuntu dot com

--- Comment #3 from Gunnar Hjalmarsson <gunnarhj at ubuntu dot com> ---
I have seen several bug reports requesting that many Latinamerican Spanish
speaking countries ought to have the 12 hour a.m./p.m. time format enabled.
Since this is not applicable to Spain, copying LC_TIME from es_ES may not be
the right thing to do in all cases.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[akhilesh.k at samsung dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

--- Comment #4 from Akhilesh Kumar <akhilesh.k at samsung dot com> ---
> ... therefore copy "es_ES" everywhere (where applicable, of course) including es_BO.

I also think copying LC_TIME from es_ES may not good, Because week set for 
es_CL, es_CU,  es_EC  and es_BO is same but different for es_ES.


week 7;19971130;4  >> es_ES 
week 7;19971130;1  >> es_CL
week 7;19971130;1  >> es_CU
week 7;19971130;1  >> es_EC
week 7;19971130;1  >> es_BO

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[maiku.fabian at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Mike FABIAN <maiku.fabian at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |maiku.fabian at gmail dot com

--- Comment #5 from Mike FABIAN <maiku.fabian at gmail dot com> ---
(In reply to Akhilesh Kumar from comment #4)
> > ... therefore copy "es_ES" everywhere (where applicable, of course) including es_BO.
> 
> I also think copying LC_TIME from es_ES may not good, Because week set for 
> es_CL, es_CU,  es_EC  and es_BO is same but different for es_ES.
> 
> 
> week 7;19971130;4  >> es_ES 
> week 7;19971130;1  >> es_CL
> week 7;19971130;1  >> es_CU
> week 7;19971130;1  >> es_EC
> week 7;19971130;1  >> es_BO

At the moment, es_CL, es_CU, es_EC, and es_BO have identical LC_TIME,
so it is OK to copy from one of them.

It would be interesting to check why MINWEEKLEN  is 4 in es_ES
and 1 in es_CL, es_CU, es_EC, and es_BO. Is this correct or is
it a mistake.

At the moment it is OK to use the suggested copy, I think, 
it shows more clearly that LC_TIME is identical in these 4 locales
at the moment.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

--- Comment #6 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  81be4b5e38c22bfd77ec8fe6815a45782c19eca1 (commit)
       via  9d5cfd8e838da31bd8fe0a55d4b43e8425dd3c2f (commit)
       via  6c7269f31dc780f27dfc0706f9b1a6dbd160fdd9 (commit)
      from  4cdc25a4a1b49c7c73124952511fb57c5a82cd3f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=81be4b5e38c22bfd77ec8fe6815a45782c19eca1

commit 81be4b5e38c22bfd77ec8fe6815a45782c19eca1
Author: Mike FABIAN <mfabian@redhat.com>
Date:   Tue Feb 20 18:14:43 2018 +0100

    Remove --quiet argument when installing locales

    Using this argument hides problems. I would like to see when something
fails.

        * localedata/Makefile: Remove --quiet argument when
        installing locales

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=9d5cfd8e838da31bd8fe0a55d4b43e8425dd3c2f

commit 9d5cfd8e838da31bd8fe0a55d4b43e8425dd3c2f
Author: Mike FABIAN <mfabian@redhat.com>
Date:   Wed Feb 14 18:17:42 2018 +0100

    Use / instead of - in d_fmt for pt_BR and pt_PT [BZ #17438]

        [BZ #17438]
        * localedata/locales/pt_BR (LC_TIME): use / instead of -
        in d_fmt.
        * localedata/locales/pt_PT (LC_TIME): likewise

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=6c7269f31dc780f27dfc0706f9b1a6dbd160fdd9

commit 6c7269f31dc780f27dfc0706f9b1a6dbd160fdd9
Author: Mike FABIAN <mfabian@redhat.com>
Date:   Tue Feb 13 12:03:49 2018 +0100

    Use “copy "es_BO"” in LC_TIME of es_CU, es_CL, and es_EC

    LC_TIME in these 4 locales is identical, using “copy "es_BO"” makes
    that more obvious.

        [BZ #22646]
        * localedata/locales/es_CL (LC_TIME): copy "es_BO".
        * localedata/locales/es_CU (LC_TIME): copy "es_BO".
        * localedata/locales/es_EC (LC_TIME): copy "es_BO".

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                |   19 +++++++++++++++++++
 localedata/Makefile      |    2 +-
 localedata/locales/es_CL |   37 +------------------------------------
 localedata/locales/es_CU |   37 +------------------------------------
 localedata/locales/es_EC |   37 +------------------------------------
 localedata/locales/pt_BR |    2 +-
 localedata/locales/pt_PT |    2 +-
 7 files changed, 25 insertions(+), 111 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[digitalfreak at lingonborough dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

--- Comment #7 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Thank you, Mike. Yes, I was wrong here, that the list of month and weekday
names is the same is not enough, LC_TIME is also date formats, 12h/24h clock,
etc. Therefore we should not just copy es_ES everywhere and merging just those
4 locales is correct.

Shall we close this bug as fixed in 2.28?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[maiku.fabian at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Mike FABIAN <maiku.fabian at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |2.28

--- Comment #8 from Mike FABIAN <maiku.fabian at gmail dot com> ---
(In reply to Rafal Luzynski from comment #7)
> Thank you, Mike. Yes, I was wrong here, that the list of month and weekday
> names is the same is not enough, LC_TIME is also date formats, 12h/24h
> clock, etc. Therefore we should not just copy es_ES everywhere and merging
> just those 4 locales is correct.
> 
> Shall we close this bug as fixed in 2.28?

Oh, yes, I forgot that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[maiku.fabian at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Mike FABIAN <maiku.fabian at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #9 from Mike FABIAN <maiku.fabian at gmail dot com> ---
Fixed

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/22646] redundant data (LC_TIME) for es_CL, es_CU, es_EC and es_BO

[jeremip11 at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=22646

Jeremi <jeremip11 at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeremip11 at gmail dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.