Fixing source of ubuntu script

facebookincubator · Mar 6, 2024 · 5c9abcd · 5c9abcd
1 parent ed6284c
commit 5c9abcd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/scheduled.yml b/.github/workflows/scheduled.yml
@@ -18,6 +18,7 @@ on:
   pull_request:
     paths:
       - ".github/workflows/scheduled.yml"
+      - scripts/*
 
   schedule:
     - cron: '0 3 * * *'
@@ -75,7 +76,7 @@ jobs:
           ref: "${{ inputs.ref || 'main' }}"
 
       - name: "Install dependencies"
-        run: cd  velox && source ./scripts/setup-ubuntu.sh
+        run: cd  velox && sudo ./scripts/setup-ubuntu.sh 
 
       - name: "Build"
         run: |
@@ -125,7 +126,7 @@ jobs:
           ref: "${{ inputs.ref || 'main' }}"
 
       - name: "Install dependencies"
-        run: source ./scripts/setup-ubuntu.sh
+        run: sudo ./scripts/setup-ubuntu.sh
 
       - name: Download presto fuzzer
         uses: actions/download-artifact@v3
@@ -173,7 +174,7 @@ jobs:
           ref: "${{ inputs.ref || 'main' }}"
 
       - name: "Install dependencies"
-        run: source ./scripts/setup-ubuntu.sh
+        run: sudo ./scripts/setup-ubuntu.sh
 
       - name: Download spark expression fuzzer
         uses: actions/download-artifact@v3
@@ -247,7 +248,7 @@ jobs:
           ref: "${{ inputs.ref || 'main' }}"
 
       - name: "Install dependencies"
-        run: source ./scripts/setup-ubuntu.sh
+        run: sudo ./scripts/setup-ubuntu.sh
 
       - name: Download aggregation fuzzer
         uses: actions/download-artifact@v3
@@ -288,7 +289,7 @@ jobs:
           ref: "${{ inputs.ref || 'main' }}"
 
       - name: "Install dependencies"
-        run: source ./scripts/setup-ubuntu.sh
+        run:  sudo ./scripts/setup-ubuntu.sh
 
       - name: Download join fuzzer
         uses: actions/download-artifact@v3

diff --git a/scripts/setup-ubuntu.sh b/scripts/setup-ubuntu.sh
@@ -29,7 +29,7 @@ BOOST_VERSION=boost-1.84.0
 NPROC=$(getconf _NPROCESSORS_ONLN)
 DEPENDENCY_DIR=${DEPENDENCY_DIR:-$(pwd)}
 export CMAKE_BUILD_TYPE=Release
-SUDO=sudo
+export SUDO=sudo
 
 # Install all velox and folly dependencies.
 # The is an issue on 22.04 where a version conflict prevents glog install,