> On Oct 9, 2025, at 12:27, Jeff Davis <[email protected]> wrote: > > * If the provider is builtin and the LC_CTYPE or LC_COLLATE environment > variables aren't compatible with UTF-8, it can override those to "C". > But if --locale, --lc-ctype, or --lc-collate are specified and > incompatible, they will throw an error instead.
Overall the code looks good to me. I do have a question about the design here. Are we assuming that * if the settings come from command line options, then the user is intentionally doing that, so we throw an error * if the settings come from env, then the user might not be aware of them, so we only issue a warning? If that’s the case, I’m not fully convinced by this design. Since initdb is a one-time operation, I think it would be better to require everything to be explicit. Best regards, -- Chao Li (Evan) HighGo Software Co., Ltd. https://www.highgo.com/
