-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rp2040_hal 2.2.1 #978
rp2040_hal 2.2.1 #978
Conversation
fb5a797
to
bbb1230
Compare
I don't understand why the CI runs are still failing. It looks like it's trying to use gnat_native when this crate can only be built with gnat_arm_elf, which is listed in the dependencies. |
This is likely related to alire-project/alire#1533. Sorry for the trouble, I'll take a look ASAP. |
I can see a warning during the build of tests:
I don't know if that's the problem. |
I can reproduce this problem locally, working on the fix. |
There are two issues here: one is some mixing of github actions caches, which was causing the apparent mixing of tools. I'm working on that. The other one is the one pointed by Fabien, which seems legit, see e.g. this run: https://github.com/alire-project/alire-index/actions/runs/7681838264/job/20986832061?pr=978#step:6:825 |
bbb1230
to
46b1ec6
Compare
My bad, the warning is the issue here [1]. The problem is that the check script is trying to run the test, which is an ARM Cortex-M application for the RP2040. The scripts assumes that the tests are native applications and that [1] Aunit is rightfully in not compiled with warnings as error. |
So, IIUC, these tests are not intended for execution here, and this release can be merged? |
Correct, these tests can only run on an RP2040. I do think there is value in compiling the tests as a part of CI, but we don't have a good simulator for running them outside of real hardware. This release can be merged, imo. |
Alright, then. After 2.0 the CI script can be updated to rely only on |
No description provided.