Skip to content

Commit a3816b4

Browse files
committed
Revert "Add BB downloading for OpenBLAS (#30497)"
This reverts commit 87c18d8.
1 parent 0152373 commit a3816b4

10 files changed

+40
-210
lines changed

.travis.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,8 @@ before_install:
7676
brew tap staticfloat/julia > /dev/null;
7777
brew rm --force $(brew deps --HEAD julia);
7878
brew install -v ccache gcc gmp mpfr pcre2 staticfloat/julia/openblas-julia staticfloat/julia/suite-sparse-julia staticfloat/juliadeps/libgfortran;
79-
BUILDOPTS="-j3 USECLANG=1 USECCACHE=1 USE_BINARYBUILDER_LLVM=1 USE_BINARYBUILDER_OPENBLAS=1 BINARYBUILDER_LLVM_ASSERTS=1";
80-
BUILDOPTS="$BUILDOPTS LLVM_CONFIG=$TRAVIS_BUILD_DIR/usr/tools/llvm-config LLVM_SIZE=$TRAVIS_BUILD_DIR/usr/tools/llvm-size";
79+
BUILDOPTS="-j3 USECLANG=1 USECCACHE=1 BINARYBUILDER_TRIPLET=x86_64-apple-darwin14 BINARYBUILDER_LLVM_ASSERTS=1";
80+
BUILDOPTS="$BUILDOPTS USE_BINARYBUILDER_LLVM=1 LLVM_CONFIG=$TRAVIS_BUILD_DIR/usr/tools/llvm-config LLVM_SIZE=$TRAVIS_BUILD_DIR/usr/tools/llvm-size";
8181
BUILDOPTS="$BUILDOPTS VERBOSE=1 USE_BLAS64=0 SUITESPARSE_INC=-I$(brew --prefix suite-sparse-julia)/include FORCE_ASSERTIONS=1";
8282
BUILDOPTS="$BUILDOPTS LIBBLAS=-lopenblas LIBBLASNAME=libopenblas LIBLAPACK=-lopenblas LIBLAPACKNAME=libopenblas";
8383
for lib in SUITESPARSE BLAS LAPACK GMP MPFR LIBUNWIND; do

Make.inc

+2-1
Original file line numberDiff line numberDiff line change
@@ -245,10 +245,11 @@ INSTALL_F := $(JULIAHOME)/contrib/install.sh 644
245245
INSTALL_M := $(JULIAHOME)/contrib/install.sh 755
246246

247247
# BinaryBuilder options
248-
USE_BINARYBUILDER_OPENBLAS := 0
248+
# TODO: Autodiscover triplet
249249
USE_BINARYBUILDER_LLVM := 0
250250
# Use the Assertions build
251251
BINARYBUILDER_LLVM_ASSERTS := 0
252+
BINARYBUILDER_TRIPLET :=
252253

253254
# LLVM Options
254255
LLVMROOT := $(build_prefix)

contrib/fixup-libgfortran.sh

+5-21
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
# This file is a part of Julia. License is MIT: https://julialang.org/license
33

44
# Run as: fixup-libgfortran.sh [--verbose] <$private_libdir>
5-
FC=${FC:-gfortran}
65

76
# If we're invoked with "--verbose", create a `debug` function that prints stuff out
87
if [ "$1" = "--verbose" ] || [ "$1" = "-v" ]; then
@@ -23,7 +22,7 @@ if [ "$UNAME" = "Linux" ]; then
2322
elif [ "$UNAME" = "Darwin" ]; then
2423
SHLIB_EXT="dylib"
2524
else
26-
echo "WARNING: Could not autodetect platform type ('uname -s' = $UNAME); assuming Linux" >&2
25+
echo "WARNING: Could not autodetect platform type ('uname -s' == $UNAME); assuming Linux" >&2
2726
UNAME="Linux"
2827
SHLIB_EXT="so"
2928
fi
@@ -42,20 +41,6 @@ find_shlib()
4241
fi
4342
}
4443

45-
find_shlib_dir()
46-
{
47-
# Usually, on platforms like OSX we get full paths when linking. However,
48-
# if we are inspecting, say, BinaryBuilder-built OpenBLAS libraries, we will
49-
# only get something like `@rpath/libgfortran.5.dylib` when inspecting the
50-
# libraries. We can, as a last resort, ask `$FC` directly what the full
51-
# filepath for this library is, but only if we don't have a direct path to it:
52-
if [ $(dirname "$1") = "@rpath" ]; then
53-
dirname "$($FC -print-file-name="$(basename "$1")" 2>/dev/null)"
54-
else
55-
dirname "$1" 2>/dev/null
56-
fi
57-
}
58-
5944
# First, discover all the places where libgfortran/libgcc is, as well as their true SONAMES
6045
for lib in lapack blas openblas; do
6146
for private_libname in ${private_libdir}/lib$lib*.$SHLIB_EXT*; do
@@ -66,11 +51,10 @@ for lib in lapack blas openblas; do
6651
LIBQUADMATH_PATH=$(find_shlib "$private_libname" libquadmath)
6752

6853
# Take the directories, add them onto LIBGFORTRAN_DIRS, which we use to
69-
# search for these libraries in the future. If there is no directory, try
70-
# asking `$FC` where such a file could be found.
71-
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(find_shlib_dir $LIBGFORTRAN_PATH)"
72-
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(find_shlib_dir $LIBGCC_PATH)"
73-
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(find_shlib_dir $LIBQUADMATH_PATH)"
54+
# search for these libraries in the future.
55+
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(dirname $LIBGFORTRAN_PATH 2>/dev/null)"
56+
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(dirname $LIBGCC_PATH 2>/dev/null)"
57+
LIBGFORTRAN_DIRS="$LIBGFORTRAN_DIRS $(dirname $LIBQUADMATH_PATH 2>/dev/null)"
7458

7559
# Save the SONAMES
7660
LIBGFORTRAN_SONAMES="$LIBGFORTRAN_SONAMES $(basename "$LIBGFORTRAN_PATH")"

contrib/normalize_triplet.py

-125
This file was deleted.

contrib/windows/appveyor_build.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,15 @@ if [ "$ARCH" = x86_64 ]; then
4545
echo 'USE_BLAS64 = 1' >> Make.user
4646
echo 'LIBBLAS = -L$(JULIAHOME)/usr/bin -lopenblas64_' >> Make.user
4747
echo 'LIBBLASNAME = libopenblas64_' >> Make.user
48+
echo 'BINARYBUILDER_TRIPLET = x86_64-w64-mingw32' >> Make.user
4849
else
4950
bits=32
5051
archsuffix=86
5152
exc=sjlj
5253
echo "override MARCH = pentium4" >> Make.user
5354
echo 'LIBBLAS = -L$(JULIAHOME)/usr/bin -lopenblas' >> Make.user
5455
echo 'LIBBLASNAME = libopenblas' >> Make.user
56+
echo 'BINARYBUILDER_TRIPLET = i686-w64-mingw32' >> Make.user
5557
fi
5658
echo "override JULIA_CPU_TARGET=generic;native" >> Make.user
5759

@@ -196,7 +198,6 @@ if [ -n "$USEMSVC" ]; then
196198
else
197199
# Use BinaryBuilder
198200
echo 'USE_BINARYBUILDER_LLVM = 1' >> Make.user
199-
echo 'USE_BINARYBUILDER_OPENBLAS = 1' >> Make.user
200201
echo 'BINARYBUILDER_LLVM_ASSERTS = 1' >> Make.user
201202
echo 'override DEP_LIBS += llvm openlibm' >> Make.user
202203
export CCACHE_DIR=/cygdrive/c/ccache

deps/Makefile

-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ include $(SRCDIR)/Versions.make
1212
include $(JULIAHOME)/Make.inc
1313
include $(SRCDIR)/tools/common.mk
1414
include $(SRCDIR)/tools/git-external.mk
15-
include $(SRCDIR)/tools/bb-install.mk
1615

1716
BUILDDIR := $(BUILDDIR)$(MAYBE_HOST)
1817

deps/blas.mk

-20
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,6 @@ endif
9292
# Do not overwrite the "-j" flag
9393
OPENBLAS_BUILD_OPTS += MAKE_NB_JOBS=0
9494

95-
ifneq ($(USE_BINARYBUILDER_OPENBLAS), 1)
96-
9795
$(BUILDDIR)/$(OPENBLAS_SRC_DIR)/build-configured: $(BUILDDIR)/$(OPENBLAS_SRC_DIR)/source-extracted
9896
perl -i -ple 's/^\s*(EXTRALIB\s*\+=\s*-lSystemStubs)\s*$$/# $$1/g' $(dir $<)/Makefile.system
9997
echo 1 > $@
@@ -196,21 +194,3 @@ configure-lapack: extract-lapack
196194
compile-lapack: $(BUILDDIR)/lapack-$(LAPACK_VER)/build-compiled
197195
fastcheck-lapack: check-lapack
198196
check-lapack: $(BUILDDIR)/lapack-$(LAPACK_VER)/build-checked
199-
200-
else # USE_BINARYBUILDER_OPENBLAS
201-
202-
203-
OPENBLAS_BB_URL_BASE := https://github.com/JuliaPackaging/Yggdrasil/releases/download/OpenBLAS-v$(OPENBLAS_VER)-$(OPENBLAS_BB_REL)
204-
OPENBLAS_BB_NAME := OpenBLAS.v$(OPENBLAS_VER)-$(OPENBLAS_BB_REL)
205-
206-
$(eval $(call bb-install,openblas,OPENBLAS,true))
207-
get-lapack: get-openblas
208-
extract-lapack: extract-openblas
209-
configure-lapack: configure-openblas
210-
compile-lapack: compile-openblas
211-
fastcheck-lapack: fastcheck-openblas
212-
check-lapack: check-openblas
213-
clean-lapack: clean-openblas
214-
distclean-lapack: distclean-openblas
215-
install-lapack: install-openblas
216-
endif

deps/llvm.mk

+29-4
Original file line numberDiff line numberDiff line change
@@ -513,12 +513,37 @@ ifeq ($(USE_POLLY),1)
513513
endif
514514
endif
515515
else # USE_BINARYBUILDER_LLVM
516-
LLVM_BB_URL_BASE := https://github.com/staticfloat/LLVMBuilder/releases/download/v$(LLVM_VER)-$(LLVM_BB_REL)
516+
LLVM_BB_URL_BASE := https://github.com/staticfloat/LLVMBuilder/releases/download
517517
ifneq ($(BINARYBUILDER_LLVM_ASSERTS), 1)
518-
LLVM_BB_NAME := LLVM.v$(LLVM_VER)
518+
LLVM_BB_NAME := LLVM
519519
else
520-
LLVM_BB_NAME := LLVM.asserts.v$(LLVM_VER)
520+
LLVM_BB_NAME := LLVM.asserts
521521
endif
522+
LLVM_BB_NAME := $(LLVM_BB_NAME).v$(LLVM_VER)
523+
LLVM_BB_URL := $(LLVM_BB_URL_BASE)/v$(LLVM_VER)-$(LLVM_BB_REL)/$(LLVM_BB_NAME).$(BINARYBUILDER_TRIPLET).tar.gz
522524

523-
$(eval $(call bb-install,llvm,LLVM,false))
525+
526+
$(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL):
527+
mkdir -p $@
528+
529+
$(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)/LLVM.$(BINARYBUILDER_TRIPLET).tar.gz: | $(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)
530+
$(JLDOWNLOAD) $@ $(LLVM_BB_URL)
531+
532+
$(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)/build-compiled: | $(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)/LLVM.$(BINARYBUILDER_TRIPLET).tar.gz
533+
echo 1 > $@
534+
535+
$(eval $(call staged-install,llvm,llvm-$$(LLVM_VER)-$$(LLVM_BB_REL),,,,))
536+
537+
#Override provision of stage tarball
538+
$(build_staging)/llvm-$(LLVM_VER)-$(LLVM_BB_REL).tgz: $(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)/LLVM.$(BINARYBUILDER_TRIPLET).tar.gz | $(build_staging)
539+
cp $< $@
540+
541+
clean-llvm:
542+
distclean-llvm:
543+
get-llvm: $(BUILDDIR)/llvm-$(LLVM_VER)-$(LLVM_BB_REL)/LLVM.$(BINARYBUILDER_TRIPLET).tar.gz
544+
extract-llvm:
545+
configure-llvm:
546+
compile-llvm:
547+
fastcheck-llvm:
548+
check-llvm:
524549
endif # USE_BINARYBUILDER_LLVM

deps/openblas.version

-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,2 @@
11
OPENBLAS_BRANCH=v0.3.3
22
OPENBLAS_SHA1=fd8d1868a126bb9f12bbc43b36ee30d1ba943fbb
3-
OPENBLAS_VER=0.3.3
4-
OPENBLAS_BB_REL=0

deps/tools/bb-install.mk

-33
This file was deleted.

0 commit comments

Comments
 (0)