acassis commented on code in PR #18661: URL: https://github.com/apache/nuttx/pull/18661#discussion_r3022398075
########## tools/espressif/burn_flash_enc_key.sh: ########## @@ -0,0 +1,82 @@ +#!/usr/bin/env bash +# tools/espressif/burn_flash_enc_key.sh +# Review Comment: @simbit18 I agree that Python is more multi-platform, but we need to decide to include it as dependence. Greg was already against adding more dependencies to the building system. I'm not against it, I'm just saying we need to decide (and vote) to add python as requirement, we need to analyze the pros and cons of adding this dependence. -- 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]
