Skip to content
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

gh: try to fix zephyr project action #47

Merged
merged 8 commits into from
Jan 24, 2025
Merged

Conversation

facchinm
Copy link
Member

No description provided.

@facchinm
Copy link
Member Author

facchinm commented Jan 24, 2025

Fixes #8 and #44

Still some issues with the overlay name resolution, will fix when the full llext-edk automatization is released
@facchinm
Copy link
Member Author

🎉 CI is back 🙂

@facchinm facchinm merged commit 205688d into arduino:arduino Jan 24, 2025
4 checks passed
@KurtE
Copy link

KurtE commented Jan 24, 2025

@facchinm - I am in the process of changing the analogRead code as mentioned.

I updated my Arduino branch to be up to date, I then went to my PR branch and rebased it to the current stuff
and pushed it up.

The build step failed:

/opt/toolchains/zephyr-sdk-0.17.0/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/12.2.0/../../../../arm-zephyr-eabi/bin/ld.bfd: zephyr/libzephyr.a(zephyrCommon.cpp.obj): in function analogWrite': /__w/ArduinoCore-zephyr/ArduinoCore-zephyr/ArduinoCore-zephyr/build/modules/lib/ArduinoCore-zephyr/cores/arduino/zephyrCommon.cpp:325: undefined reference to map(long, long, long, long, long)'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: /usr/bin/cmake --build /__w/ArduinoCore-zephyr/ArduinoCore-zephyr/ArduinoCore-zephyr/build/build

It said that map function was undefined... Will be pushing up changes to code later, will be testing soon.

@facchinm
Copy link
Member Author

Just fixed that bug too via 205bd78 🙂 I'll restart the CI on your branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants