mciantyre / teensy4-rs

Rust support for the Teensy 4
Apache License 2.0
271 stars 31 forks source link

Make `imxrt-log` an optional dependency #156

Closed NZRosto closed 4 months ago

NZRosto commented 4 months ago

Hi all,

This is a small PR in an effort to reduce unused dependencies in my project tree. For people (like me) that roll their own USB logging / serial communication, making imxrt-log optional reduces the size of the dependency tree a lot.

I don't have much experience with the inner workings of this project, so this is my best effort at guessing what it would take to make this work. Let me know what needs to change and I'll do my best to update this!

As a side note: I wasn't sure why the rtic_usb_log example didn't already have usb logging as a required feature, but it didn't build until I added it, if that's wrong or intentional please let me know.

Thanks for all the hard work! Really looking forward to developing on teensy with Rust.