Page MenuHomeFreeBSD

Add Farsi/Persian locales
AcceptedPublic

Authored by kfv_irbug.org on Apr 10 2020, 1:07 AM.

Details

Summary

Add fa_AF.UTF-8 and fa_IR.UTF-8 locales

Diff Detail

Repository
rS FreeBSD src repository
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 30441
Build 28199: arc lint + arc unit

Event Timeline

kfv_irbug.org created this revision.Apr 10 2020, 1:07 AM

Removed numericdef/fa_AF.UTF-8.src and numericdef/ar_SA.UTF-8.src since they are same with numericdef/fa_IR.UTF-8.src

This comment was removed by kfv_irbug.org.

Remove unnecessary files that were set the same as another file in Makefile

0mp resigned from this revision.Apr 22 2020, 8:06 PM
0mp edited reviewers, added: Contributor Reviews (base); removed: 0mp.

I'm sorry but I don't know how the locale generation system works + I do not have a src bit to get it committed.

lwhsu added a reviewer: bapt.Apr 22 2020, 8:12 PM
lwhsu added subscribers: bapt, lwhsu.

I believe @bapt knows this. :-)

lwhsu added a subscriber: farrokhi.

@mmokhi @farrokhi : can you help verify the correctness about Farsi/Persian? Thanks!

farrokhi requested changes to this revision.Jun 11 2020, 2:24 PM
farrokhi added inline comments.
share/numericdef/Makefile
12

Should these be removed? Aren't you only adding fa_IR?

share/timedef/fa_IR.UTF-8.src
7

We do not have Hamza in persian alphabet. It should be written is ژانویه. The same goes for the rest of words ending with Hamza in this set.

This revision now requires changes to proceed.Jun 11 2020, 2:24 PM
kfv_irbug.org added inline comments.Jun 11 2020, 5:48 PM
share/numericdef/Makefile
12

Yes, because of two things:

  1. They are the same, so it makes sense to have either ar_SA.UTF-8 or fa_IR.UTF-8.
  2. These are not manual changes, I've just followed /usr/src/tools/tools/locale and made the diff.

I can use the tool one more time to see if it produces anything else.

share/timedef/fa_IR.UTF-8.src
7

Completely true, but we should either add a manual change or contribute to the CLDR project on cldr.unicode.org.
Anyhow, I will do all steps one more time and will update you.

Although I think nothing's wrong with this revision since all steps are automatically done and I've made no additional change, I believe @yuripv knows much more than me about locales. So I would appreciate if he could also take a look.

share/numericdef/Makefile
12

OK, I just double-checked and we've done nothing wrong here. It removes the ar_SA.UTF-8 from LOCALES+= and adds the fa_IR.UTF-8 instead. The rest are taken care of in SAME+= lines.

share/timedef/fa_IR.UTF-8.src
7

I redid all steps one more time and nothing's change. I will check how this could be fixed on CLDR, but I think we are fine for this revision and there is nothing to work on. But, there's something interesting that I forgot to tell you before, timedef is commented out in /usr/src/tools/tools/locale/Makefile - check line 20:

KNOWN=          monetdef numericdef msgdef colldef ctypedef # timedef

So I had to uncomment it to generate them. So maybe, it wouldn't cause any problem if we manually remove all occurrences of the trailing Hamza. Am I right? Let me know if we can do so and I will apply the change.

@farrokhi: is this version good?

This revision is now accepted and ready to land.Tue, Jul 21, 11:51 AM