(polaris) branch main updated: Do not use relative path inside CLI script

[em...@apache.org]

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

emaynard pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 2d3d0d0dd Do not use relative path inside CLI script
2d3d0d0dd is described below

commit 2d3d0d0dd8b3812a630e0ac9abc8b8f1a4b469d0
Author: Eric Maynard <er...@snowflake.com>
AuthorDate: Sat Jun 14 09:51:44 2025 -0700

    Do not use relative path inside CLI script
    
    Issue #1868 reported that the Polaris script can fail when it's run from an unexpected path. The recent addition of a reference to `./gradlew` looks incorrect here, and should be changed to use an absolute path.
    
    Fixes #1868
---
 polaris | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/polaris b/polaris
index 70736fbd1..a8fb55837 100755
--- a/polaris
+++ b/polaris
@@ -32,14 +32,14 @@ done
 
 # Check if the python client needs regeneration
 if [ "$repair" == true ] || [ ! -f "${dir}/client/python/polaris/catalog/__init__.py" ]; then
-  ./gradlew regeneratePythonClient
+  "${dir}/gradlew" regeneratePythonClient
 fi
 
 if [ ! -d "${dir}"/polaris-venv ] || [ "$repair" == true ]; then
   rm -f "${dir}"/poetry.lock
   rm -f "${dir}/client/python/poetry.lock"
 
-  if [ ! -d ${dir}/polaris-venv ]; then
+  if [ ! -d "${dir}/polaris-venv" ]; then
     echo "Performing first-time setup for the Python client..."
     python3 -m venv "${dir}"/polaris-venv
   else