Skip to content

Commit 4540b9f

Browse files
chouetzofek
andauthored
fix(install): Another try to have a proper argument split (#56)
* fix(install): Another try to have a proper argument split * use alternative method to parse features * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review --------- Co-authored-by: Ofek Lev <ofekmeister@gmail.com>
1 parent 1133022 commit 4540b9f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

main.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,14 @@ install_features() {
2727
if [[ -n "${FEATURES}" ]]; then
2828
echo -e "${PURPLE}Installing features: ${FEATURES}${RESET}"
2929

30+
# Temporarily change IFS to space just for this loop
31+
IFS=' ' read -ra features <<< "$FEATURES"
3032
ARGS=()
31-
for feature in $(echo $FEATURES); do
33+
for feature in features; do
3234
ARGS+=("-f" "$feature")
3335
done
3436

35-
"${1}" -v self dep sync "${ARGS[@]}"
37+
"${1}" -v self dep sync ${ARGS[@]}
3638
fi
3739
}
3840

0 commit comments

Comments
 (0)