introduce C.UTF-8 locale
ClosedPublic
Actions

Authored by yuripv on Nov 4 2018, 6:30 AM.

Details

Reviewers

bapt
eadler
cem

Commits

rS341627: MFC r340144:
rS340144: Add hybrid C.UTF-8 locale being identical to default C locale except

Summary

Introduce C.UTF-8 locale, using the same common CTYPE map as other UTF-8 locales (and it's now the source of symlinks instead of en_US one), and having all other components use C locale.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

yuripv created this revision.Nov 4 2018, 6:30 AM

Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptNov 4 2018, 6:30 AM

cem added a subscriber: cem.Nov 4 2018, 6:39 AM

bapt accepted this revision.Nov 4 2018, 6:50 AM

This revision is now accepted and ready to land.Nov 4 2018, 6:50 AM

review is for everything but the perl code. That seems okay but I don't trust my review :)

0mp added a subscriber: 0mp.Nov 4 2018, 1:22 PM

emaste added a subscriber: emaste.Nov 4 2018, 5:24 PM

I really like the logical change — thank you for making it! I don't have total familiarity with this area, especially the Perl, but nothing about it looks bad to me.

Just for the note: perl bits here are only the helper tools for someone updating the locale sources in share/, and aren't run during the build, so even if something is broken there (and I tried not to as I used them), it's only bad until there's a need to update the data again where they will be fixed anyway :)

Closed by commit rS340144: Add hybrid C.UTF-8 locale being identical to default C locale except (authored by yuripv). · Explain WhyNov 4 2018, 10:13 PM

This revision was automatically updated to reflect the committed changes.