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