Skip to content

Add installation of locales for Icinga Web - #496

Open
Donien wants to merge 1 commit into
mainfrom
fix/475-locales
Open

Add installation of locales for Icinga Web#496
Donien wants to merge 1 commit into
mainfrom
fix/475-locales

Conversation

@Donien

@Donien Donien commented Aug 21, 2026

Copy link
Copy Markdown
Member

Fixes #475

@chrnie chrnie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice feature, thanks for picking up #475. I went through the review points and pushed the result as #497, which targets this branch rather than main - merging it there lands everything here.

The substance is in how locale identifiers turn into packages. The RedHat langpacks are split by language, not by territory, so the territory has to be dropped entirely: de_DE and de_AT both are glibc-langpack-de.
The old regex kept everything up to the last underscore, which made sr_RS_latin resolve to glibc-langpack-sr_RS, and an entry without an underscore did not match at all, so a bare de reached dnf as a package literally named de.
Both fail with a package-not-found error naming a string the user never wrote, so identifiers are now validated up front against _ and refused with a message that says what the expected form is.

The README also contradicted itself - the prose asked for . with a dot while the example below it used de_DE. And locale_gen gets one call per locale now, since it only accepts a
list from community.general 9.3.0 onwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Install system locales in icingaweb host

2 participants