simbit18 commented on PR #17236:
URL: https://github.com/apache/nuttx/pull/17236#issuecomment-3451427183

   @TangMeng12 , allow me to make a few observations. 
   
   As already mentioned, the current system was designed to speed up and reduce 
errors during the cloning/download step (also possible with Curl downloads if 
desired) on our CI.
   
   This system on CI makes sense when the packages need to be used in multiple 
configurations  
   
   jobs Linux (risc-v-04), Linux (xtensa-01), Linux (xtensa-02) 
   
   With this simple check
   ifneq ($(wildcard $(NXTMPDIR)/.*),)
   
   We can insert any package (if necessary) from the Nuttx or Apps repository 
into the storage directory.
   
   Storage is enabled with
     
   -S adds the nxtmpdir folder for third-party packages.
   
    in the build.yml workflow on GitHub
   
https://github.com/apache/nuttx/blob/201406b22b8035763fe81752aad71a1fba394ee8/.github/workflows/build.yml#L194
   
   and if desired, you can use it locally 
   example
   ./tools/configure.sh -l -S esp32-devkitc:nsh
   
   
   At the moment, simply using CONFIG_ALLOW_DOWNLOADS (must be added to all 
Expressif configurations!) does not meet the requirements of our CI!
   


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

Reply via email to