[GLUTEN-10635][VL] bugfix: file INSTALL cannot set permissions

[beliefer]

What changes are proposed in this pull request?

This PR proposes to fix a building bug.
Fixes #10635

How was this patch tested?

GA tests.

[github-actions]

#10635

[Zouxxyy]

I actually encountered this problem when running build_arrow on my Mac, and I was confused as to why arrow was installed in a fixed location like this cmake --install ./ --prefix /usr/local/. Instead of using INSTALL_PREFIX

if [[ "$(uname)" == "Darwin" ]]; then
    INSTALL_PREFIX=${INSTALL_PREFIX:-${VELOX_HOME}/deps-install}
else
    INSTALL_PREFIX=${INSTALL_PREFIX:-"/usr/local"}
fi

Maybe we need a fix? @zhouyuan

[philo-he]

@Zouxxyy, MacOS was not properly considered in the Arrow installation. You can propose a fix to use a path more suitable for MacOS. Maybe, you can just replace /usr/local with $INSTALL_PREFIX in build_arrow.sh.

[philo-he]

Can we just let user fix this issue by setting SUDO=sudo in the environment?

[beliefer]

Thank you! I tried in this way. But it is still not working.

[zhouyuan]

Can you pease try to add this line in the beginning of this file?
https://github.com/apache/incubator-gluten/blob/main/dev/build_arrow.sh#L21

[beliefer]

Done.

[philo-he]

It seems line 21 should be changed to SUDO="${SUDO:-""}", where all letters are capitalized. Then, I assume setting SUDO=sudo in the environment should work.

[beliefer]

It seems line 21 should be changed to SUDO="${SUDO:-""}", where all letters are capitalized. Then, I assume setting SUDO=sudo in the environment should work.

I tried this way, it works well. But I don't know if it will affect other OS.

[philo-he]

@beliefer, I suggest to just fix this line, and let users explicitly set the environment variable if they meet such permission issue.

[Zouxxyy]

@Zouxxyy, MacOS was not properly considered in the Arrow installation. You can propose a fix to use a path more suitable for MacOS. Maybe, you can just replace /usr/local with $INSTALL_PREFIX in build_arrow.sh.

Sure, can you have a look at #10650, thanks

[philo-he]

Thanks.

[beliefer]

@philo-he @zhouyuan Thank you!