Skip to content

Add a test for the defmt feature - #577

Open
TDecking wants to merge 1 commit into
servo:v2from
TDecking:defmt
Open

Add a test for the defmt feature#577
TDecking wants to merge 1 commit into
servo:v2from
TDecking:defmt

Conversation

@TDecking

@TDecking TDecking commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Closes #475.

This test does a lot less than one might expect because defmt takes up a lot of responsibility for correctness.

The official documentation (https://defmt.ferrous-systems.com/faq-for-library-authors) recommends to use QEMU to
run a cross compiled binary for testing. I decided not to do that. Instead, I used a crate for connecting defmt with the log crate, which is IMHO a better apprach.

Because defmt is designed for embedded systems, this test won't run on either Windows or with miri. Windows is not supported by defmt, and the program tries to access certain linker sections of its binary, which runs into issues with miri's isolation.

@alejandro-vaz

Copy link
Copy Markdown
Collaborator

is it worth it to upgrade to 1.88 just for those kinds of let expressions??

two version = three months, which puts us at a +2.5 month surplus from the original baseline

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.

defmt testing

2 participants