fix: better error handling when build module

fzf-tab.zsh

@@ -343,7 +343,16 @@ toggle-fzf-tab() {
 }
 
 build-fzf-tab-module() {
-  -ftb-build-module $@
+  {
+    pushd -q $FZF_TAB_HOME/modules
+    if -ftb-build-module $@; then
+      print -P "%F{green}%BThe module has been built successfully. Please restart zsh to apply it.%f%b"
+    else
+      print -P -u2 "%F{red}%BThe module building has failed. See the output above for details.%f%b"
+    fi
+  } always {
+    popd -q
+  }
 }
 
 zmodload zsh/zutil

lib/-ftb-build-module

@@ -1,5 +1,5 @@
 #!/hint/zsh
-emulate -LR zsh -o extended_glob
+emulate -LR zsh -o extended_glob -o err_exit
 
 local zsh_version=${1:-${FZF_TAB_ZSH_SRC_VERSION:-$ZSH_VERSION}}
 
@@ -12,8 +12,6 @@ else
   nproc=$(nproc)
 fi
 
-pushd -q $FZF_TAB_HOME/modules
-
 # clone zsh source code if not exists
 if [[ ! -d ./zsh/$zsh_version ]]; then
   git clone --depth=1 --branch zsh-$zsh_version https://github.com/zsh-users/zsh ./zsh/$zsh_version
@@ -30,16 +28,6 @@ if [[ ! -f ./Makefile ]]; then
   ./configure --disable-gdbm --disable-pcre --without-tcsetpgrp --prefix=/tmp/zsh-fzf-tab-module ${bundle:+DL_EXT=bundle}
 fi
 make -j$nproc
-ret=$?
-
-if (( $ret != 0 )); then
-  print -P -u2 "%F{red}%BThe module building has failed. See the output above for details.%f%b"
-  return $ret
-fi
 
 # we only need aloxaf/fzftab.so
 mv ./Src/Modules/fzftab.(so|bundle) $FZF_TAB_HOME/modules/Src/aloxaf/
-
-popd -q
-
-print -P "%F{green}%BThe module has been built successfully. Please restart zsh to apply it.%f%b"