Skip to content

Commit d9bdc90

Browse files
committed
💅 RUBY_CONFIGURE_OPTS_ARRAY
1 parent fbdf88c commit d9bdc90

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

bin/ruby-build

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -529,6 +529,7 @@ build_package() {
529529
package_option() {
530530
local package_name="$1"
531531
local command_name="$2"
532+
# e.g. RUBY_CONFIGURE_OPTS_ARRAY, OPENSSL_MAKE_OPTS_ARRAY
532533
local variable="$(capitalize "${package_name}_${command_name}")_OPTS_ARRAY"
533534
local array="$variable[@]"
534535
shift 2
@@ -650,18 +651,18 @@ build_package_ruby() {
650651
build_package_ree_installer() {
651652
build_package_auto_tcltk
652653

653-
local options=""
654-
is_mac && options="--no-tcmalloc"
654+
local options=()
655+
is_mac && options+=(--no-tcmalloc)
655656

656657
local option
657-
for option in ${RUBY_CONFIGURE_OPTS_ARRAY[@]} $RUBY_CONFIGURE_OPTS; do
658-
options="$options -c $option"
658+
for option in "${RUBY_CONFIGURE_OPTS_ARRAY[@]}" $RUBY_CONFIGURE_OPTS; do
659+
options+=(-c "$option")
659660
done
660661

661662
# Work around install_useful_libraries crash with --dont-install-useful-gems
662663
mkdir -p "$PREFIX_PATH/lib/ruby/gems/1.8/gems"
663664

664-
{ ./installer --auto "$PREFIX_PATH" --dont-install-useful-gems $options $CONFIGURE_OPTS
665+
{ ./installer --auto "$PREFIX_PATH" --dont-install-useful-gems "${options[@]}" $CONFIGURE_OPTS
665666
} >&4 2>&1
666667
}
667668

0 commit comments

Comments
 (0)