unknowntpo commented on code in PR #6848:
URL: https://github.com/apache/gravitino/pull/6848#discussion_r2032467710


##########
dev/docker/ranger/ranger-dependency.sh:
##########
@@ -36,7 +36,36 @@ if [[ ! -d "${ranger_dir}/packages" ]]; then
 fi
 
 if [ ! -f "${ranger_dir}/packages/${RANGER_PACKAGE_NAME}" ]; then
-  curl -L -s -o "${ranger_dir}/packages/${RANGER_PACKAGE_NAME}" 
${RANGER_DOWNLOAD_URL}
+  # Package not exist, we need to build them from source
+       if [ ! -d "${ranger_dir}/packages/apache-ranger" ]; then
+               git clone https://github.com/apache/ranger --branch master 
--single-branch ${ranger_dir}/packages/apache-ranger
+               # set the commit to RANGER-5146: 500 API Error When Deleting 
TagDef with a Linked Tag
+               # 
https://github.com/apache/ranger/commit/ff36aabe36169b94862c51a5b403f59c9d728b94
+               git reset --hard ff36aabe36169b94862c51a5b403f59c9d728b94
+       fi
+
+       cp ${ranger_dir}/.env 
${ranger_dir}/packages/apache-ranger/dev-support/ranger-docker
+
+       cd ${ranger_dir}/packages/apache-ranger/dev-support/ranger-docker

Review Comment:
   Ok, I removed the new line between two `cp` commands.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@gravitino.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to