Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Cc: "Michał Górny" <mgorny@g.o>
Subject: [gentoo-dev] [PATCH 1/7] ninja-utils.eclass: Split get_NINJAOPTS out
Date: Fri, 25 Nov 2022 17:05:48
Message-Id: 20221125170508.23064-2-mgorny@gentoo.org
In Reply to: [gentoo-dev] [PATCH 0/7] distutils-r1.eclass: mesonpy option passing support + periodic cleanup by "Michał Górny"
1 Signed-off-by: Michał Górny <mgorny@g.o>
2 ---
3 eclass/ninja-utils.eclass | 15 +++++++++++----
4 1 file changed, 11 insertions(+), 4 deletions(-)
5
6 diff --git a/eclass/ninja-utils.eclass b/eclass/ninja-utils.eclass
7 index 0dffd2eb86ff..9be502fa8ad9 100644
8 --- a/eclass/ninja-utils.eclass
9 +++ b/eclass/ninja-utils.eclass
10 @@ -62,6 +62,16 @@ case "${NINJA}" in
11 ;;
12 esac
13
14 +# @FUNCTION: get_NINJAOPTS
15 +# @DESCRIPTION:
16 +# Get the value of NINJAOPTS, inferring them from MAKEOPTS if unset.
17 +get_NINJAOPTS() {
18 + if [[ -z ${NINJAOPTS+set} ]]; then
19 + NINJAOPTS="-j$(makeopts_jobs "${MAKEOPTS}" 999) -l$(makeopts_loadavg "${MAKEOPTS}" 0)"
20 + fi
21 + echo "${NINJAOPTS}"
22 +}
23 +
24 # @FUNCTION: eninja
25 # @USAGE: [<args>...]
26 # @DESCRIPTION:
27 @@ -72,11 +82,8 @@ eninja() {
28 local nonfatal_args=()
29 [[ ${EAPI} != 5 ]] && nonfatal_args+=( -n )
30
31 - if [[ -z ${NINJAOPTS+set} ]]; then
32 - NINJAOPTS="-j$(makeopts_jobs "${MAKEOPTS}" 999) -l$(makeopts_loadavg "${MAKEOPTS}" 0)"
33 - fi
34 [[ -n "${NINJA_DEPEND}" ]] || ewarn "Unknown value '${NINJA}' for \${NINJA}"
35 - set -- "${NINJA}" -v ${NINJAOPTS} "$@"
36 + set -- "${NINJA}" -v $(get_NINJAOPTS) "$@"
37 echo "$@" >&2
38 "$@" || die "${nonfatal_args[@]}" "${*} failed"
39 }
40 --
41 2.38.1