search

espressif / esp-idf-ci-action

GitHub Action for ESP32 CI
MIT License
59 stars 24 forks source link

Mount the Repository under its name instead of app (RDT-315) #24

Closed Fishwaldo closed 1 year ago

Fishwaldo commented 1 year ago

I'm using this to build some examples in a component I'm developing As per the developer documentation for the components, we should use a override_path in the idf_component.yml file for each example: https://github.com/espressif/idf-component-manager/wiki#add-examples-to-the-component

The problem is that the base path is mounted as "app" and not the component name, so when idf was trying to resolve dependencies it was failing.

This change, mounts the code under the repository name instead of app, and this idf can resolve it using the path_override.

(maybe a better option would be to provide a config option to specify the path the repository is mounted under in the docker image?)

kumekay commented 1 year ago

@Fishwaldo Thank you for the improvement