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/03/15 12:34:52 UTC

[GitHub] [airflow] potiuk commented on pull request #14787: Fixes recent scripting breeze fix to work also with zsh

potiuk commented on pull request #14787:
URL: https://github.com/apache/airflow/pull/14787#issuecomment-799383530


   True, in theory it should - https://tldp.org/LDP/abs/html/internalvariables.html, however I've learned not to trust MacOS environment and be a little defensive (I am not 100% sure what was the scenario that led to BASH env missing - but taking into account the large variations of Bash's there for MacOS (the built-in 3.2, any version that can be installed by brew etc.) - `command -v bash` is all but guaranteed to work. 
   
   But yeah - it might be an interesting question how it got there.


----------------------------------------------------------------
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:
users@infra.apache.org