Source verification ensures using the original user in tmp directory (#…

…110) Closes GH-108
apache · Sep 11, 2023 · 849661e · 849661e
1 parent e4233a7
commit 849661e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 18 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -46,23 +46,6 @@ jobs:
       - name: Prepare
         run: |
           ci/scripts/prepare-${{ matrix.os }}.sh ${{ matrix.postgresql-version }}
-      - name: Install Apache Arrow Flight SQL adapter
-        run: |
-          setup_args=(
-            "--prefix=/tmp/local"
-            "-Dbenchmark=true"
-            "-Dpostgresql_dir=$(pg_config --bindir)/.."
-          )
-          if [ ${{ matrix.os }} = "macos" ]; then
-            setup_args+=("--pkg-config-path=$(brew --prefix libpq)/lib/pkgconfig")
-          fi
-          meson setup "${setup_args[@]}" build
-          meson compile -C build
-          if [ ${{ matrix.os }} = "ubuntu" ]; then
-            sudo meson install -C build
-          else
-            meson install -C build
-          fi
       - uses: ruby/setup-ruby@v1
         with:
           ruby-version: ruby

diff --git a/dev/release/verify-rc.sh b/dev/release/verify-rc.sh
@@ -203,7 +203,7 @@ test_source_distribution() {
   fi
 
   meson setup \
-        --prefix="${PWD}/install" \
+        --prefix="${ARROW_TMPDIR}/install" \
         -Dpostgresql_dir="$(pg_config --bindir)/.." \
         ${ARROW_FLIGHT_SQL_POSTGRESQL_MESON_SETUP_ARGS:-} \
         build
@@ -227,6 +227,7 @@ test_source_distribution() {
       Linux-/usr/*)
         if type sudo > /dev/null 2>&1; then
           sudo meson install -C build
+          sudo chown -R "${USER}:" "${ARROW_TMPDIR}/install"
         else
           meson install -C build
         fi