tmedicci commented on issue #18568:
URL: https://github.com/apache/nuttx/issues/18568#issuecomment-4134371532

   > Hi [@tmedicci](https://github.com/tmedicci), I understand your request, 
and we could consider adding another workflow for Espressif.
   
   I don't think we need something specifically for Espressif. We can set any 
environment var to the build system. There are a couple of them that "tweaks" 
the build system.
   
   > However, if we add the option to modify the ESP_HAL_3RDPARTY_VERSION 
environment variable, all of these architectures
   > 
   >     * arch/risc-v
   > 
   >     * arch/risc-v/src/esp32c3-legacy
   > 
   >     * arch/xtensa/src/esp32
   > 
   >     * arch/xtensa/src/esp32s2
   > 
   >     * arch/xtensa/src/esp32s3
   > will use the same Git hash, but as far as I can see, that is not currently 
the case, so some jobs will fail.
   
   There are a couple of things to consider here: 
   1. `ESP_HAL_3RDPARTY_VERSION` isn't usually set because the PR itself 
already contains the version being tested. Just setting `ESP_HAL_3RDPARTY_URL` 
is enough to test an under-development branch (yet not yet published, for 
instance).
   2. However, we always keep it aligned: when we update the version, we do 
that for every supported chip (xtensa update is on the way). An exception to 
that is `esp32c3-legacy`, which is expected to be removed in a major NuttX 
release.
   


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