search

OpenAMP / open-amp

The main OpenAMP library implementing RPMSG, Virtio, and Remoteproc for RTOS etc
https://www.openampproject.org/
Other
721 stars 291 forks source link

CI: Zephyr, test against known working version and also the very latest on main #623

Closed wmamills closed 1 month ago

wmamills commented 1 month ago

This PR brings open-amp up to parity with libmetal with regard to Zephyr CI testing

open-amp did not have the latest SDK discovery code nor the sdk install reduction of verbosity that were already in libmetal so this PR is a bit bigger than the one for libmetal but the end result is the same.

wmamills commented 1 month ago

@arnopo I have marked this as RFC until your PR goes in. I will then rebase and repush it and remove the RFC. This PR should look the same but I will remove any merge conflicts with yours.

wmamills commented 1 month ago

@arnopo I rebased this on your CI update. As yours is not in main yet you will see your commits in this PR but they should disappear after you merge your PR. (Let me know if there is any issue with that.)

I took the "draft" mark off this now. It should be ready to go as soon as you accept your PR.

I went ahead and clones openamp-system-reference for the zephyr jobs to keep things consistent. The zephyr build step does not use the openamp-system-reference directory right now but it may in the future.

wmamills commented 1 month ago

Rebased and retested. Merge should be OK