search

rust-embedded / embedonomicon

How to bootstrap support for a no_std target
https://docs.rust-embedded.org/embedonomicon/
Apache License 2.0
206 stars 33 forks source link

Update objdump outputs #88

Closed adamgreig closed 2 years ago

adamgreig commented 2 years ago

Hopefully this fixes CI, let's see.

rust-highfive commented 2 years ago

r? @andre-richter

(rust-highfive has picked a reviewer for you, use r? to override)

adamgreig commented 2 years ago

Yay. I removed the beta and stable builds and bumped the MSRV 1.58 to 1.62 because something changed in the output format between those versions. I think it makes sense to just build for a specific version and for nightly rather than all four from before, to help reduce CI breakages for unrelated PRs.

It's worth noting that the latest qemu-system-arm I tried (6.2) breaks with this, on an error about "Timer with period zero, disabling". I don't know what's caused that but I got past it by downloading the same pre-built qemu that the CI uses.

bors[bot] commented 2 years ago

Canceled.

adamgreig commented 2 years ago

The nightly output broke overnight :sob: and I forgot to tell bors about the change in Rust version. Hopefully it sticks this time.

bors[bot] commented 2 years ago

Build succeeded: