-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Cargo 0.28.0 and later break cargo-bloat for embedded #5955
Comments
Thanks for the report! We don't provide stability guarantees though about Cargo used through its library API. To that end we're happy to take PRs but we don't track the issues like this here unless it's a bug with Cargo-the-CLI |
So this is it then? A valuable tool breaks, the author can't fix the problem because it's in a third party component and the authors of a third party component won't even look at the problem? Frustrating... |
As a solution going forward, perhaps it's worth having a CI build (either during PR validation and/or a nightly build) that tests against latest/source cargo? The idea being that you would have forewarning when a breaking change lands. Maybe it should be a best practice for cargo extensions (custom commands, or whatnot). |
It's certainly a good idea to do that but I don't see how it addresses the problem. Say you're forewarned about an upcoming compatibility issue, what are the options? Staying on the old version indefinitely? And then let me ask the same question again but with the background that the tool we're talking about just so happened to work for embedded development without any intention to enable that use case in the first place. |
As reported in RazrFalcon/cargo-bloat#25 I upgraded
cargo-bloat
to the latest version which also bumps thecargo
dependency to 0.28.0 and this seemingly breakscargo bloat
for use on cross-compiled binaries. I verified that bumping the dependency to 0.29.0 does not address the issue while going back to 0.27.0 rectifies the problem.The text was updated successfully, but these errors were encountered: