[GitHub] [airflow] uranusjr edited a comment on pull request #19867: Initial commit for new Breeze project

[GitBox <gi...@apache.org>]

uranusjr edited a comment on pull request #19867:
URL: https://github.com/apache/airflow/pull/19867#issuecomment-981700207


   > This is not what the documentation tells: [docs.python.org/3/library/os.html#os.execv](https://docs.python.org/3/library/os.html#os.execv)
   > 
   > | Availability: Unix, Windows.
   
   I was inaccurate. The command works, but probably won’t do what you want 🙂 
   
   [BPO-19124](https://bugs.python.org/issue19124) (and the linked [BPO-19066](https://bugs.python.org/issue19066)) has some detailed information on this. Quoting some dire but accurate comments:
   
   > [C]ontrol is returned to cmd when the child process *starts* (and afterwards you have cmd and the child connected to the same console).
   
   > This is why I said that execv() is useless on Windows and that you should just use subprocess instead.
   
   > The exec functions provided by the Windows C runtime really are practically useless, due to creating an orphaned process, disrupting synchronous operation, and returning the wrong status code.


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