[gentoo-commits] proj/locale-gen:master commit in: /

[Mike Frysinger]

commit:     953486683088c5aed6af94e97644a64c68e8d1f5
Author:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
AuthorDate: Mon Sep 27 06:38:45 2021 +0000
Commit:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
CommitDate: Mon Sep 27 06:38:45 2021 +0000
URL:        https://gitweb.gentoo.org/proj/locale-gen.git/commit/?id=95348668

fix hint about use of / in locale names

We checked $ret outside of the subshell where it was set, so this
code never actually fired.  Move it up to where it should work.

Signed-off-by: Mike Frysinger <vapier <AT> gentoo.org>

 locale-gen | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/locale-gen b/locale-gen
index 03caff2..c746eef 100755
--- a/locale-gen
+++ b/locale-gen
@@ -299,14 +299,14 @@ generate_locale() {
 		elif [[ ${ret} -ne 0 ]] ; then
 			eerror "${disp}: ${x}"
 		fi
+
+		if [[ ${ret} -ne 0 && ${locale} == */* ]] ; then
+			ewarn "Perhaps you meant to use a space instead of a / in your config file ?"
+		fi
 		exit ${ret}
 	) &
 	JOB_PIDS+=( $! )
 	: $(( ++JOB_IDX_E ))
-
-	if [[ ${ret} != 0 && ${locale} == */* ]] ; then
-		ewarn "Perhaps you meant to use a space instead of a / in your config file ?"
-	fi
 }
 
 JOB_PIDS=()