Bug 23996 - Dutch salutations
Summary: Dutch salutations
Status: RESOLVED FIXED
Alias: None
Product: glibc
Classification: Unclassified
Component: localedata (show other bugs)
Version: unspecified
: P2 minor
Target Milestone: 2.30
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-12-16 20:09 UTC by Pander
Modified: 2019-07-17 10:14 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Last reconfirmed: 2019-07-10 00:00:00
fweimer: security-


Attachments
Patch adding Dutch salutations (261 bytes, patch)
2018-12-16 20:09 UTC, Pander
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pander 2018-12-16 20:09:07 UTC
Created attachment 11465 [details]
Patch adding Dutch salutations

Please, add the following salutations for Dutch locales for Netherlands and Belgium:

name_gen    "heer of mevrouw"
name_mr     "heer"
name_mrs    "mevrouw"
name_miss   "mevrouw"
name_ms     "mevrouw"

The attached git patch will offer this for the files nl_NL and nl_BE in glibc/localedata/locales
Comment 1 Rafal Luzynski 2018-12-31 18:21:36 UTC
1. Do I see correctly that LC_NAME in nl_BE is the same as nl_NL?  Can we use “copy "nl_NL"” in nl_BE instead?

2. I think it's a mistake to set this bug as depending on deprecation of LC_NAME.
Comment 2 Pander 2019-01-01 11:42:04 UTC
Yes, I agree with you that it is not depending on deprecation of LC_NAME, as the content will find another destination along with the other LC_NAME definitions.
Comment 3 Pander 2019-01-01 11:53:34 UTC
And yes, nl_BE can use the values from nl_NL via a copy as the content is identical for nl_BE and nl_NL.

Looking at LC_TIME in nl_BE, that can also use copy "nl_NL" Not sure why this was commented out and literally copied from nl_NL. So to save space, this can be fixed by using this copy will have the benefit of also adding date_fmt to nl_BE. Does this require another bug report or can that be covered by this one?

Good to know: The official Dutch language is identical for nl_BE and nl_NL. This applies also to all official formats for representing salutations, dates, times, monetary issues, etc.
Comment 4 Rafal Luzynski 2019-07-10 09:55:02 UTC
Thank you, Pander. Since CLDR does not support salutations I can just trust you. Here is your patch posted again:

https://sourceware.org/ml/libc-alpha/2019-07/msg00215.html

and my additional fixup:

https://sourceware.org/ml/libc-alpha/2019-07/msg00216.html

Hopefully they will be accepted and pushed soon.
Comment 5 Pander 2019-07-10 09:58:54 UTC
Thanks!
Comment 6 Sourceware Commits 2019-07-17 10:07:06 UTC
The master branch has been updated by Rafal Luzynski <rl@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=3cc7c9c5f1b47c891f08daf18394e90b756a88b8

commit 3cc7c9c5f1b47c891f08daf18394e90b756a88b8
Author: PanderMusubi <pander@users.sourceforge.net>
Date:   Sun Dec 16 20:09:07 2018 +0000

    nl_BE and nl_NL locales: Dutch salutations (bug 23996).
    
    	[BZ #23996]
    	* localedata/locales/nl_BE (LC_NAME): Add name_gen, name_mr,
    	name_mrs, name_miss, and name_ms.
    	* localedata/locales/nl_NL (LC_NAME): Likewise.
Comment 7 Sourceware Commits 2019-07-17 10:07:10 UTC
The master branch has been updated by Rafal Luzynski <rl@sourceware.org>:

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

commit fba6d4bbce716b8256cb33f06ec8ae607a4556ef
Author: Rafal Luzynski <digitalfreak@lingonborough.com>
Date:   Tue Jul 9 11:53:43 2019 +0200

    nl_BE locale: Use "copy "nl_NL"" in LC_NAME (bug 23996).
    
    The content of the section is identical in both languages.
    
    	[BZ #23996]
    	* localedata/locales/nl_BE (LC_NAME): Replace with “copy "nl_NL"”.
Comment 8 Rafal Luzynski 2019-07-17 10:14:39 UTC
This bug has been fixed in master.  Thank you Pander for your contribution.