Skip to content

Commit 2e2305e

Browse files
committed
workflows/l10n.yml: Fix l10n_installation_test.
In "Test Make installation" step, set french locale BEFORE building with make. In "Test Cargo installation" step, add french locale env var before installation. If not, the binary is built with no embedded french locale.
1 parent 7bd08d4 commit 2e2305e

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

.github/workflows/l10n.yml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -595,6 +595,10 @@ jobs:
595595
## Test installation via make with DESTDIR
596596
echo "Testing make install with l10n features..."
597597
598+
# Set French locale
599+
export LANG=fr_FR.UTF-8
600+
export LC_ALL=fr_FR.UTF-8
601+
598602
# Create installation directory
599603
MAKE_INSTALL_DIR="$PWD/make-install-dir"
600604
mkdir -p "$MAKE_INSTALL_DIR"
@@ -632,10 +636,6 @@ jobs:
632636
if [ "${{ matrix.job.os }}" = "ubuntu-latest" ]; then
633637
echo "Testing French localization with make-installed binary..."
634638
635-
# Set French locale
636-
export LANG=fr_FR.UTF-8
637-
export LC_ALL=fr_FR.UTF-8
638-
639639
echo "Testing ls --help with French locale..."
640640
ls_help=$("$MAKE_INSTALL_DIR/usr/bin/coreutils" ls --help 2>&1 || echo "Command failed")
641641
echo "ls help output (first 10 lines):"
@@ -697,6 +697,10 @@ jobs:
697697
## Test installation via cargo install with DESTDIR-like approach
698698
echo "Testing cargo install with l10n features..."
699699
700+
# Set French locale
701+
export LANG=fr_FR.UTF-8
702+
export LC_ALL=fr_FR.UTF-8
703+
700704
# Create installation directory
701705
CARGO_INSTALL_DIR="$PWD/cargo-install-dir"
702706
mkdir -p "$CARGO_INSTALL_DIR"

0 commit comments

Comments
 (0)