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



##########
File path: scripts/in_container/_in_container_utils.sh
##########
@@ -263,14 +263,16 @@ function install_airflow_from_wheel() {
     echo
     echo "Install airflow wheel package from dist"
     echo
-    pip install /dist/apache_airflow-*.whl >"${OUT_FILE_PRINTED_ON_ERROR}" 2>&1
+    pip install /dist/apache_airflow-*.whl --no-deps 
>"${OUT_FILE_PRINTED_ON_ERROR}" 2>&1

Review comment:
       Yep. I can add a comment, but this is easy to explain. By that time all 
the dependencies are already installed (we remove only the packages (airflow + 
airflow-providers) that were installed from sources, but their dependencies are 
already installed as needed as we installed 'devel_ci' extra. That's why we 
just need to install the packages, build and nothing else.




----------------------------------------------------------------
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.

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


Reply via email to