No locale fallback specified for fr and es

[brechtm]

I would assume fr-CL to fallback to fr-FR and es-CL to es-ES, but these are not listed in the table in the CSL spec.

[rmzelle]

Thanks. Looks like we should add "es-MX" and "es-CL" as secondary dialects with "es-ES" as primary for Spanish, and "fr-CA" as secondary dialect with "fr-FR" as primary dialect for French.

As long as we don't have a dedicated "fr-CL" CSL locale file, we wouldn't add it to the fallback table in the CSL spec, though. Do you have users asking for it?

(and in general, while I think it's good to formalize fallback locales in the spec, in most cases it's rather obvious which locale should be primary and which are secondary, so I don't think it's crucial to keep the CSL spec table all-inclusive)

[rmzelle]

(or did you mistype "fr-CA" as "fr-CL"?)

[rmzelle]

Oh, and it might not be documented anywhere, but https://github.com/citation-style-language/locales/blob/master/locales.json should define all primary locales.

[brechtm]

Yes, I meant "fr-CA".

Thanks for the pointer on locales.json. I suppose I can count on that file being kept up to date? I'll use that instead of defining the mappings in my source code. Perhaps it's a good idea to mention this file in the CSL specification docs?

[rmzelle]

We don't get new locales very often (the last was in 2015), but yeah, we have automatic Travis CI testing to make sure that locales.json is valid JSON and defines a primary dialect for all languages in the locales repo, so it should be pretty robust (https://github.com/citation-style-language/locales/blob/master/spec/repository_spec.rb).

[rmzelle]

Perhaps it's a good idea to mention this file in the CSL specification docs?

Yes, or somewhere else in our project documentation.

[rmzelle]

(I'll leave this open so I won't forgot to update the fallback table in the spec)

[customcommander]

@rmzelle @brechtm Please see PR #56