Commit e8dc5ac
committed
tools: limit parallelism with ninja in V8 builds
If `JOBS` has been set in the environment to limit build parallelism,
pass that onto `ninja` when building V8.
PR-URL: #51473
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>1 parent e51221b commit e8dc5ac
1 file changed
Lines changed: 7 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -9,6 +9,11 @@ cd deps/v8 || exit | |||
| 9 | 9 | find . -type d -name .git -print0 | xargs -0 rm -rf | |
| 10 | 10 | ../../tools/v8/fetch_deps.py . | |
| 11 | 11 | ||
| 12 | + JOBS_ARG= | ||
| 13 | + if [ "${JOBS}" ]; then | ||
| 14 | + JOBS_ARG="-j ${JOBS}" | ||
| 15 | + fi | ||
| 16 | + | ||
| 12 | 17 | ARCH=$(arch) | |
| 13 | 18 | if [ "$ARCH" = "s390x" ] || [ "$ARCH" = "ppc64le" ]; then | |
| 14 | 19 | TARGET_ARCH=$ARCH | |
@@ -46,10 +51,10 @@ if [ "$ARCH" = "s390x" ] || [ "$ARCH" = "ppc64le" ]; then | |||
| 46 | 51 | gcc --version | |
| 47 | 52 | export PKG_CONFIG_PATH=$BUILD_TOOLS/pkg-config | |
| 48 | 53 | gn gen -v "out.gn/$BUILD_ARCH_TYPE" --args="is_component_build=false is_debug=false use_goma=false goma_dir=\"None\" use_custom_libcxx=false v8_target_cpu=\"$TARGET_ARCH\" target_cpu=\"$TARGET_ARCH\" v8_enable_backtrace=true $CC_WRAPPER" | |
| 49 | - ninja -v -C "out.gn/$BUILD_ARCH_TYPE" d8 cctest inspector-test | ||
| 54 | + ninja -v -C "out.gn/$BUILD_ARCH_TYPE" "${JOBS_ARG}" d8 cctest inspector-test | ||
| 50 | 55 | else | |
| 51 | 56 | DEPOT_TOOLS_DIR="$(cd _depot_tools && pwd)" | |
| 52 | 57 | # shellcheck disable=SC2086 | |
| 53 | 58 | PATH="$DEPOT_TOOLS_DIR":$PATH tools/dev/v8gen.py "$BUILD_ARCH_TYPE" --no-goma $V8_BUILD_OPTIONS | |
| 54 | - PATH="$DEPOT_TOOLS_DIR":$PATH ninja -C "out.gn/$BUILD_ARCH_TYPE/" d8 cctest inspector-test | ||
| 59 | + PATH="$DEPOT_TOOLS_DIR":$PATH ninja -C "out.gn/$BUILD_ARCH_TYPE/" "${JOBS_ARG}" d8 cctest inspector-test | ||
| 55 | 60 | fi | |
0 commit comments