[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=()