summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKazuhiro NISHIYAMA <znz@users.noreply.github.com>2019-08-26 21:09:02 +0900
committerGitHub <noreply@github.com>2019-08-26 21:09:02 +0900
commitccc5b22a7f4302a286cb16a797e8dccf422e8279 (patch)
tree3ef5fbc1cb0a656bcacc10c2c2f509882027113c
parent48f1a38f23f9a544e71c5656c83c419c064509fd (diff)
Expand JOBS instead of meaningless export (#2402)
* Expand JOBS instead of meaningless export * Remove -j option in TESTOPTS of test-bundled-gems https://github.com/ruby/ruby/runs/203129516#step:10:167 > invalid option: -j3
Notes
Notes: Merged-By: znz
-rw-r--r--.github/workflows/ubuntu.yml10
1 files changed, 3 insertions, 7 deletions
diff --git a/.github/workflows/ubuntu.yml b/.github/workflows/ubuntu.yml
index 1c82c576c5..7b58b32d07 100644
--- a/.github/workflows/ubuntu.yml
+++ b/.github/workflows/ubuntu.yml
@@ -34,9 +34,6 @@ jobs:
GITHUB_BRANCH: ${{ github.event.pull_request.head.ref }}
GITHUB_REPO: ${{ github.event.pull_request.head.repo.full_name }}
if: github.event_name == 'pull_request'
- - name: Set ENV
- run: |
- export JOBS=-j$((1 + $(nproc --all)))
- name: Fixed world writable dirs
run: |
chmod go-w $HOME
@@ -44,18 +41,17 @@ jobs:
- run: autoconf
- name: configure
run: ./configure -C --disable-install-doc
- - run: make $JOBS
+ - run: make -j$((1 + $(nproc --all)))
- name: Tests
- run: make -s ${{ matrix.test_task }}
+ run: make -s TESTOPTS="-j$((1 + $(nproc --all))) -q --tty=no" ${{ matrix.test_task }}
env:
- TESTOPTS: "$JOBS -q --tty=no"
MSPECOPT: "-ff" # not using `-j` because sometimes `mspec -j` silently dies
if: matrix.test_task != 'test-bundled-gems' && !contains(github.event.head_commit.message, '[ci skip]')
# test-bundled-gems is separated for marking `continue-on-error` because it randomly fails.
- name: Tests
run: make -s ${{ matrix.test_task }}
env:
- TESTOPTS: "$JOBS -q --tty=no"
+ TESTOPTS: "-q --tty=no"
continue-on-error: true
if: matrix.test_task == 'test-bundled-gems' && !contains(github.event.head_commit.message, '[ci skip]')
- name: Leaked Globals