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