New review comment by sgn on void-packages repository https://github.com/void-linux/void-packages/pull/26035#discussion_r515561249 Comment: I would try to avoid nested `if .. then ..fi` I guess something like: ```sh if [ -z "$CHROOT_READY" ]; then hostmakedepends="cmake-bootstrap" unset XBPS_CHECK_PKGS else hostmakedepends="asciidoc cmake perl" fi if [ -n "$XBPS_CHECK_PKGS" ]; then configure_args+=" -DENABLE_TESTING=ON" else configure_args+=" -DENABLE_TESTING=OFF" fi ``` The `unset` is also used in `binutils`. I'll try a full bootstrap later.