potiuk commented on a change in pull request #20678:
URL: https://github.com/apache/airflow/pull/20678#discussion_r778973216



##########
File path: scripts/docker/install_additional_dependencies.sh
##########
@@ -36,26 +35,26 @@ function install_additional_dependencies() {
         echo
         echo Installing additional dependencies while upgrading to newer 
dependencies
         echo
-        pip install ${AIRFLOW_INSTALL_USER_FLAG} --upgrade --upgrade-strategy 
eager \
+        pip install --upgrade --upgrade-strategy eager \
             ${ADDITIONAL_PYTHON_DEPS} ${EAGER_UPGRADE_ADDITIONAL_REQUIREMENTS}
         # make sure correct PIP version is used
-        pip install ${AIRFLOW_INSTALL_USER_FLAG} --upgrade 
"pip==${AIRFLOW_PIP_VERSION}"

Review comment:
       Apparently looks like I even added comment about it :)
   ```
   # By default PIP will install everything in ~/.local
   ARG PIP_USER="true"
   ```




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to