You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/07 14:44:24 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #19992: Enable local Breeze script and pipx to be used for breeze bootstrap

potiuk edited a comment on pull request #19992:
URL: https://github.com/apache/airflow/pull/19992#issuecomment-987986808


   @uranusjr  - I changed the name of the PR to reflect the dual nature of running PIP. I also added autoamated detection of where the airflow sources are:
   
   1) First we look in the working directory (upwards)
   2) If we are outside of working directory of Airflow, we look into where it was installed from (upwards)
   
   This does not support the case that we want to run (potentially) different Breeze depending on where we run it (which our bootstrap script does) but it handles a different case (which `pipx` is actually better at) that Breeze2 might be run from any directory, even outside of Airflow code and it will work `as if` it was run from withing the repo.
   
   @edithturn - we are close to merging it, I think :)
   


-- 
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-unsubscribe@airflow.apache.org

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