support libyaml from homebrew

bin/ruby-build

@@ -580,6 +580,7 @@ build_package_standard_build() {
 
   if [ "$package_var_name" = "RUBY" ]; then
       use_homebrew_readline || use_freebsd_pkg ||true
+      use_homebrew_yaml || true
   fi
 
   ( if [ "${CFLAGS+defined}" ] || [ "${!PACKAGE_CFLAGS+defined}" ]; then
@@ -1022,17 +1023,18 @@ locate_llvm() {
 }
 
 needs_yaml() {
-  [[ "$RUBY_CONFIGURE_OPTS" != *--with-libyaml-dir=* ]] &&
   ! use_homebrew_yaml
 }
 
 use_homebrew_yaml() {
-  local libdir="$(brew --prefix libyaml 2>/dev/null || true)"
-  if [ -d "$libdir" ]; then
-    echo "ruby-build: using libyaml from homebrew"
-    package_option ruby configure --with-libyaml-dir="$libdir"
-  else
-    return 1
+  if [[ "$RUBY_CONFIGURE_OPTS" != *--with-libyaml-dir=* ]]; then
+    local libdir="$(brew --prefix libyaml 2>/dev/null || true)"
+    if [ -d "$libdir" ]; then
+      echo "ruby-build: using libyaml from homebrew"
+      package_option ruby configure --with-libyaml-dir="$libdir"
+    else
+      return 1
+    fi
   fi
 }
 

test/build.bats

@@ -186,10 +186,12 @@ OUT
   mkdir -p "$brew_libdir"
 
   stub uname '-s : echo Linux'
-  stub brew "--prefix libyaml : echo '$brew_libdir'" false
+  stub brew false "--prefix libyaml : echo '$brew_libdir'"
   stub_make_install
 
-  install_fixture definitions/needs-yaml
+  run_inline_definition <<DEF
+install_package "ruby-2.0.0" "http://ruby-lang.org/ruby/2.0/ruby-2.0.0.tar.gz"
+DEF
   assert_success
 
   unstub uname
@@ -209,7 +211,7 @@ OUT
   readline_libdir="$TMP/homebrew-readline"
   mkdir -p "$readline_libdir"
 
-  stub brew "--prefix readline : echo '$readline_libdir'"
+  stub brew "--prefix readline : echo '$readline_libdir'" false
   stub_make_install
 
   run_inline_definition <<DEF
@@ -230,7 +232,7 @@ OUT
 @test "readline is not linked from Homebrew when explicitly defined" {
   cached_tarball "ruby-2.0.0"
 
-  stub brew
+  stub brew false
   stub_make_install
 
   export RUBY_CONFIGURE_OPTS='--with-readline-dir=/custom'