diff options
author | Mike Frysinger <vapier@gentoo.org> | 2021-09-27 02:38:45 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2021-09-27 02:38:45 -0400 |
commit | 953486683088c5aed6af94e97644a64c68e8d1f5 (patch) | |
tree | 02c7be6bba024e2d4e5c396fc6f1bd339913e727 | |
parent | add quotes around : ${...} defaults (diff) | |
download | locale-gen-953486683088c5aed6af94e97644a64c68e8d1f5.tar.gz locale-gen-953486683088c5aed6af94e97644a64c68e8d1f5.tar.bz2 locale-gen-953486683088c5aed6af94e97644a64c68e8d1f5.zip |
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@gentoo.org>
-rwxr-xr-x | locale-gen | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -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=() |