[apisix] branch master updated: chore: add sudo privilege for make install command at installing luarocks (#4708)

[tokers]

This is an automated email from the ASF dual-hosted git repository.

tokers pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git


The following commit(s) were added to refs/heads/master by this push:
     new d96a0f9  chore: add sudo privilege for make install command at 
installing luarocks (#4708)
d96a0f9 is described below

commit d96a0f9c74f1690b841c5cd099f7b57c771da84c
Author: Joey <majun...@apache.org>
AuthorDate: Fri Jul 30 09:58:19 2021 +0800

    chore: add sudo privilege for make install command at installing luarocks 
(#4708)
    
    
    Signed-off-by: imjoey <majun...@apache.org>
---
 .github/workflows/code-lint.yml   | 2 +-
 .github/workflows/fuzzing-ci.yaml | 2 +-
 utils/linux-install-luarocks.sh   | 3 +--
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/code-lint.yml b/.github/workflows/code-lint.yml
index 78f853b..be995b4 100644
--- a/.github/workflows/code-lint.yml
+++ b/.github/workflows/code-lint.yml
@@ -18,7 +18,7 @@ jobs:
         export OPENRESTY_VERSION=default
 
         ./utils/linux-install-openresty.sh
-        sudo ./utils/linux-install-luarocks.sh
+        ./utils/linux-install-luarocks.sh
         sudo luarocks install luacheck
 
     - name: Script
diff --git a/.github/workflows/fuzzing-ci.yaml 
b/.github/workflows/fuzzing-ci.yaml
index 43de19d..a161835 100644
--- a/.github/workflows/fuzzing-ci.yaml
+++ b/.github/workflows/fuzzing-ci.yaml
@@ -53,7 +53,7 @@ jobs:
         sudo add-apt-repository -y "deb http://openresty.org/package/ubuntu 
$(lsb_release -sc) main"
         sudo apt-get update
         sudo apt-get install -y git openresty curl openresty-openssl111-dev 
unzip make gcc
-        sudo ./utils/linux-install-luarocks.sh
+        ./utils/linux-install-luarocks.sh
 
         make deps
         make init
diff --git a/utils/linux-install-luarocks.sh b/utils/linux-install-luarocks.sh
index 8f0fd79..6a6d6b4 100755
--- a/utils/linux-install-luarocks.sh
+++ b/utils/linux-install-luarocks.sh
@@ -39,8 +39,7 @@ fi
     > build.log 2>&1 || (cat build.log && exit 1)
 
 make build > build.log 2>&1 || (cat build.log && exit 1)
-msg="rerun this script with 'sudo' if you failed to make install because of 
privilege problem."
-make install > build.log 2>&1 || (cat build.log && echo "$msg" && exit 1)
+sudo make install > build.log 2>&1 || (cat build.log && exit 1)
 cd .. || exit
 rm -rf luarocks-3.4.0