archlinux / archinstall

Arch Linux installer - guided, templates etc.
GNU General Public License v3.0
5.84k stars 508 forks source link

Locale generator fixes #2422

Closed C0rn3j closed 2 months ago

C0rn3j commented 4 months ago

PR Description:

Unify tab usage instead of mixing tabs and spaces.

Write variables better to not erroneously use a wrong variable, also quote them to handle filenames with whitespace better.

Finally, exit on error instead of ignoring issues.

Note that this breaks the current git master, as msgfmt has a fatal error in one of the language mutations.
That's an easy fix, but it's something to fix in the language files, not in the script.

Tests and Checks

svartkanin commented 4 months ago

I wish there was a code style that we can automatically apply 😓