locale "nl" not supported by collation plugin

[389-ds-bot]

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/54

• Created at 2012-01-04 19:26:45 by mkosek
• Closed as Fixed
• Assigned to nhosoi (@nhosoi)
• Associated bugzillas
  • https://bugzilla.redhat.com/show_bug.cgi?id=697082

---

https://bugzilla.redhat.com/show_bug.cgi?id=697082

There seems to be some problem with nl and nl_BG - the collation plugin says
they are not supported.  We need to investigate this.

[389-ds-bot]

Comment from rmeggins (@richm) at 2012-08-14 19:57:05

set default ticket origin to Community

[389-ds-bot]

Comment from nkinder (@nkinder) at 2012-08-28 04:14:47

Added initial screened field value.

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2012-12-21 07:14:46

"ldap/schema/slapd-collations.conf" has nl and nl-BE:

collation nl "" "" 1 3  2.16.840.1.113730.3.3.2.33.1    nl (Dutch)
collation nl BE "" 1 3  2.16.840.1.113730.3.3.2.34.1    nl-BE (Dutch Belgium)

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2013-10-03 00:04:12

Bug description: In the recent version of ICU, some locales do not
have its specific collator, but are included in the default (root)
locale. "nl", "en", and "fr" are in the class. ICU API ucol_open
takes the locale string and returns the collator with the status.
If the locale has no dedicated collator and the root collator is
picked up, status U_USING_DEFAULT_WARNING is returned, which is not
an error. But collation_indexer_create (collate.c) treats it as an
error and stops the collation.

Fix description: As ICU doc suggests, error checking for ucol_open
is replaced with "(U_SUCCESS(err)", by which the status U_USING_
DEFAULT_WARNING is correctly handled.

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2013-10-03 00:04:52

git patch file (master)
0001-Ticket-54-locale-nl-not-supported-by-collation-plugi.patch

[389-ds-bot]

Comment from rmeggins (@richm) at 2013-10-03 00:28:56

good catch!

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2013-10-03 03:22:11

Reviewed by Rich (Thank you!!)

Pushed to master:
5862d6c..c8d1cf5 master -> master
commit c8d1cf5

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2017-02-11 22:59:17

Metadata Update from @nhosoi:

• Issue assigned to nhosoi
• Issue set to the milestone: 1.3.2 - 09/13 (September)