Skip to content

Commit

Permalink
Fix USE_BINARYBUILDER_OPENBLAS=0 on macOS (JuliaLang#42538)
Browse files Browse the repository at this point in the history
* Fix USE_BINARYBUILDER_OPENBLAS=0 on macOS

We didn't properly move the OpenBLAS -> Objconv dependency chain
over to manifest files.  This was not visible until now because we
rarely build OpenBLAS from source without also building Objconv from
source.

Fixes JuliaLang#42519

* Also update path to `objconv` binary
  • Loading branch information
staticfloat authored and LilithHafner committed Mar 8, 2022
1 parent b1240eb commit 5966372
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions deps/openblas.mk
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ endif
ifeq ($(USE_BLAS64), 1)
OPENBLAS_BUILD_OPTS += INTERFACE64=1 SYMBOLSUFFIX="$(OPENBLAS_SYMBOLSUFFIX)" LIBPREFIX="libopenblas$(OPENBLAS_LIBNAMESUFFIX)"
ifeq ($(OS), Darwin)
OPENBLAS_BUILD_OPTS += OBJCONV=$(abspath $(BUILDDIR)/objconv/objconv)
$(BUILDDIR)/$(OPENBLAS_SRC_DIR)/build-compiled: | $(BUILDDIR)/objconv/build-compiled
OPENBLAS_BUILD_OPTS += OBJCONV=$(abspath $(build_bindir)/objconv)
$(BUILDDIR)/$(OPENBLAS_SRC_DIR)/build-compiled: | $(build_prefix)/manifest/objconv
endif
endif

Expand Down

0 comments on commit 5966372

Please sign in to comment.