- Notifications
You must be signed in to change notification settings - Fork143
releases/mcap-cli-v0.0.51
- releases/mcap-cli-v0.0.51
fe9dc10
Compare
Choose a tag to compare Could not load tagsNothing to show{{ refName }}defaultLoading
- releases/mcap-cli-v0.0.51
fe9dc10
Compare
Choose a tag to compare Could not load tagsNothing to show{{ refName }}defaultLoading
### Changelog<!-- Write a one-sentence summary of the user-impacting change (API,UI/UX, performance, etc) that could appear in a changelog. Write "None"if there is no user-facing change -->cli: fixes error when input mcap file uses lz4 block checksums### Docs<!-- Link to a Docs PR, tracking ticket in Linear, OR write "None" if nodocumentation changes are needed. -->None### DescriptionAs noted in #1294 MCAP files created in Rust were failing when runthrough the MCAP CLI because of an invalid block checksum check in thelz4 decoder. This was fixed upstream so this change just updates thedependency.I've tested using `mcap doctor` and `mcap cat` on a file with thecorrect checksum and it passes with this change and fails without.<!-- Describe the problem, what has changed, and motivation behind thosechanges. Pretend you are advocating for this change and the reader isskeptical. --><!-- In addition to unit tests, describe any manual testing you did tovalidate this change. --><table><tr><th>Before</th><th>After</th></tr><tr><td><!--before content goes here--></td><td><!--after content goes here--></td></tr></table><!-- If necessary, link relevant Linear or Github issues. Use `Fixes:foxglove/repo#1234` to auto-close the Github issue or Fixes: FG-### forLinear isses. -->
Assets2
Uh oh!
There was an error while loading.Please reload this page.