Skip to content

Commit ad8a0cf

Browse files
xhochywesm
authored andcommitted
ARROW-739: Don't install jemalloc in parallel
Alternative fix proposal. Couldn't trigger it locally though. Author: Uwe L. Korn <[email protected]> Author: Robert Nishihara <[email protected]> Closes #456 from xhochy/ARROW-739 and squashes the following commits: c1cad56 [Robert Nishihara] Replace MAKE -> CMAKE_MAKE_PROGRAM in CMakeLists.txt. f121072 [Uwe L. Korn] Add install to install command e8803b8 [Uwe L. Korn] ARROW-739: Don't install jemalloc in parallel
1 parent 005826f commit ad8a0cf

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

cpp/CMakeLists.txt

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -637,14 +637,16 @@ if (ARROW_JEMALLOC)
637637
URL https://github.com/jemalloc/jemalloc/releases/download/${JEMALLOC_VERSION}/jemalloc-${JEMALLOC_VERSION}.tar.bz2
638638
CONFIGURE_COMMAND ./configure "--prefix=${JEMALLOC_PREFIX}" "--with-jemalloc-prefix="
639639
BUILD_IN_SOURCE 1
640-
BUILD_COMMAND ${MAKE}
641-
BUILD_BYPRODUCTS "${JEMALLOC_STATIC_LIB}" "${JEMALLOC_SHARED_LIB}")
640+
BUILD_COMMAND ${CMAKE_MAKE_PROGRAM}
641+
BUILD_BYPRODUCTS "${JEMALLOC_STATIC_LIB}" "${JEMALLOC_SHARED_LIB}"
642+
INSTALL_COMMAND ${CMAKE_MAKE_PROGRAM} -j1 install)
642643
else()
643644
ExternalProject_Add(jemalloc_ep
644645
URL https://github.com/jemalloc/jemalloc/releases/download/${JEMALLOC_VERSION}/jemalloc-${JEMALLOC_VERSION}.tar.bz2
645646
CONFIGURE_COMMAND ./configure "--prefix=${JEMALLOC_PREFIX}" "--with-jemalloc-prefix="
646647
BUILD_IN_SOURCE 1
647-
BUILD_COMMAND ${MAKE})
648+
BUILD_COMMAND ${CMAKE_MAKE_PROGRAM}
649+
INSTALL_COMMAND ${CMAKE_MAKE_PROGRAM} -j1 install)
648650
endif()
649651
else()
650652
set(JEMALLOC_VENDORED 0)

0 commit comments

Comments
 (0)