Skip to content

cargo build diagnostic improvements #154

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

antonok-edm
Copy link
Contributor

I use cargo objcopy to compile some projects, but the unformatted messages are challenging to scan visually. The diagnostics are also not printed if there is a build error.

For those two reasons, I often find myself falling back to cargo build instead. But that means sometimes I will only run cargo build, thus not creating a new version of the flashable binary, and losing track of whether or not my changes are doing anything.

Typing this out now, I realize that something like cargo build && cargo objcopy would probably solve most of my problems, but I think changing those two defaults would still be a big usability improvement over the current behavior of the binutils tools.

@antonok-edm antonok-edm requested a review from a team as a code owner March 27, 2025 23:16
@antonok-edm
Copy link
Contributor Author

antonok-edm commented Mar 27, 2025

CI failures are unrelated but I opened #155 to fix them as well.

that was merged 🎉

@antonok-edm antonok-edm force-pushed the build-diagnostic-improvements branch from 0ff78b4 to c65575e Compare March 28, 2025 00:31
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.

1 participant