Overview
Request 1036430 accepted
- Added zh_Hans (simplified Chinese) and zh_Hant (traditional
chinese) locales
One of the packages I maintain includes files for these locales and for that reason, not including them in filesystem gives directory ownership problems over them.
- Created by ozu
- In state accepted
- 5 package maintainers
Loading...
All other packages put their translations into the existing directories I suppose. How about moving the ones you have? The question also is whether files in those new directories would actually be used if the locale is set to zh_CN or zh_TW?
Unfortunately, I can't read written Chinese so I am unable to do any real tests. I also don't know much about Chinese localization, other than the fact that some programs take into account those locales (a quick search on Github shows that). China is very big so it makes sense that there are more locales than other countries. But aside from my personal considerations, I made this proposal because the official ICU Documentation explicitly mentions zh_Hans and zh_Hant as locales. So I think it's just cleaner to give directory ownership to the filesystem package rather than have some arbitrary non-system package take it. It's a small change that probably does not impact a lot of users but it is consistent with what is done with other locales, I think.
Is not a matter of impact but whether it makes sense for your program. If you install files into the wrong dir you could just as well not install them.
You can easily test your program and see whether it outputs chinese, eg with zypper one would do: LANG=zh_CN zypper --help
So if your program puts mo files in zh_Hans it would still show English. If you move them to zh_CN it should work.
Actually, both
LANG=zh_HansandLANG=zh_Hantdo not work with this specific program. I getGtk-WARNING **: 19:28:21.566: Locale not supported by C library. Using the fallback 'C' locale Fontconfig warning: ignoring zh_Hans: not a valid region tagThe same happens for
zh_Hant. My submission tofilesystemstands because those two seem to be actual ICU locales, though.@a_jaeger, @dimstar, @kukuk, @lnussel, @oertel: review reminder