Skip to content

set proper locale files on Debian#95

Merged
saz merged 1 commit into
masterfrom
fix-debian-locations
Oct 21, 2025
Merged

set proper locale files on Debian#95
saz merged 1 commit into
masterfrom
fix-debian-locations

Conversation

@saz

@saz saz commented Oct 20, 2025

Copy link
Copy Markdown
Owner

Currently, this module creates a symlink even on Debian 12, which shouldn't be the case. This module should match the following table (checked against fresh Debian installs):

OS /etc/locale.conf /etc/default/locale
Debian 11 absent file
Debian 12 absent file
Debian 13 file symlink to ../locale.conf

@saz

saz commented Oct 20, 2025

Copy link
Copy Markdown
Owner Author

@smortex This PR is related to #91 and is supposed to properly handle files and symlinks. Happy about a review.

@saz saz mentioned this pull request Oct 20, 2025

@smortex smortex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That looks correct.

Comment thread manifests/params.pp Outdated
@saz saz force-pushed the fix-debian-locations branch from 020132e to c2868a1 Compare October 21, 2025 09:43
@saz saz merged commit a3e81e8 into master Oct 21, 2025
4 checks passed
@saz saz deleted the fix-debian-locations branch October 21, 2025 10:09
@saz saz added the bug label Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants