-
Notifications
You must be signed in to change notification settings - Fork 32
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
Also depending on defmt_rtt causes linker error #130
Comments
Thanks for reporting. Reproduced in the BSP template project with Both
It was a poor call on my part to enable
I've effectively turned I'm looking for non-breaking ways to fix this, but I haven't found it yet. A proper fix likely requires a new release series of the BSP with changes to the dependency / features. Additionally, I might need to re-think the |
👍 Thanks again for all your work on these crates! Where I can, I'll see if I can give back in an some way. (My apologies if it ever feels like I'm passively lobbing issue reports over the fence -- I'd normally like to take a more active role, but I'm trying to wrap up some projects while I still have my funemployment time, and I figure it's better to at least open an issue (even if hastily written and a little dry) than to hold off until I have more free time, only to probably forget about it after I've implemented some work around 😅.) |
You're all good! I appreciate these issues; keep them coming. Additionally, I prioritize issues that enable you to take that active role. Send those thoughts my way, too. I want to make sure you can hit the ground running once you get that free time. |
0.5.0 of teensy4-bsp drops the |
Describe the bug
If I have
teensy4-bsp
as a dependency anddefmt_rtt
, I run into the follow linker error:If I make the following change then I can compile successfully:
To Reproduce
Build with both
teensy4-bsp
as a dependency anddefmt_rtt
.Expected behavior
Should compile successfully.
Additional context
🤔
The text was updated successfully, but these errors were encountered: