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.
set default ticket origin to Community
Added initial screened field value.
"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) }}}
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.
git patch file (master) 0001-Ticket-54-locale-nl-not-supported-by-collation-plugi.patch
good catch!
Reviewed by Rich (Thank you!!)
Pushed to master: 5862d6c..c8d1cf5 master -> master commit c8d1cf5
Metadata Update from @nhosoi: - Issue assigned to nhosoi - Issue set to the milestone: 1.3.2 - 09/13 (September)
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/54
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.