diff options
| author |   Mike Frysinger <vapier@gentoo.org> | 2017-02-09 13:15:13 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2017-02-09 13:16:07 -0500 |
| commit | 9470312c23d126f0055f82c0e656003b3945430b (patch) | |
| tree | b7f2359d9f627ac59df262213d391b7d420b3768 /eclass/tests/multiprocessing_makeopts_jobs.sh | |
| parent | dev-php/PEAR-Text_Figlet: remove unused PEAR-Text_Figlet-1.0.2.ebuild. (diff) | |
| download | gentoo-9470312c23d126f0055f82c0e656003b3945430b.tar.gz gentoo-9470312c23d126f0055f82c0e656003b3945430b.tar.bz2 gentoo-9470312c23d126f0055f82c0e656003b3945430b.zip | |
multiprocess.eclass: makeopts_{jobs,loadavg}: fix implicit handling of $MAKEOPTS #608242
We missed quoting on ${MAKEOPTS} to set it as the first arg which meant
we might load invalid values into the second arg which is the "infinite"
scenario.
Diffstat (limited to 'eclass/tests/multiprocessing_makeopts_jobs.sh')
| -rwxr-xr-x | eclass/tests/multiprocessing_makeopts_jobs.sh | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/eclass/tests/multiprocessing_makeopts_jobs.sh b/eclass/tests/multiprocessing_makeopts_jobs.sh index ef477277ab3b..cc4f91b69436 100755 --- a/eclass/tests/multiprocessing_makeopts_jobs.sh +++ b/eclass/tests/multiprocessing_makeopts_jobs.sh @@ -10,14 +10,20 @@ inherit multiprocessing test-makeopts_jobs() { local exp=$1; shift tbegin "makeopts_jobs($1${2+; inf=${2}}) == ${exp}" - local act=$(makeopts_jobs "$@") - [[ ${act} == "${exp}" ]] - tend $? "Got back: ${act}" + local indirect=$(MAKEOPTS="$*" makeopts_jobs) + local direct=$(makeopts_jobs "$@") + if [[ "${direct}" != "${indirect}" ]] ; then + tend 1 "Mismatch between MAKEOPTS/cli: '${indirect}' != '${direct}'" + else + [[ ${direct} == "${exp}" ]] + tend $? "Got back: ${act}" + fi } tests=( 999 "-j" 999 "--jobs" + 999 "-j -l9" 1 "" 1 "-l9 -w" 1 "-l9 -w-j4" |