From ccc5b22a7f4302a286cb16a797e8dccf422e8279 Mon Sep 17 00:00:00 2001
From: Kazuhiro NISHIYAMA <znz@users.noreply.github.com>
Date: Mon, 26 Aug 2019 21:09:02 +0900
Subject: [PATCH] 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
---
 .github/workflows/ubuntu.yml | 10 +++-------
 1 file 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