Skip to content
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

Release v0.4.0 #29

Merged
merged 1 commit into from
Feb 7, 2023
Merged

Release v0.4.0 #29

merged 1 commit into from
Feb 7, 2023

Conversation

robin-nitrokey
Copy link
Member

Added

  • Added Path::from_str_with_nul and path! to create Path instances from
    str.
  • Added Eq and PartialEq implementations for Path.

Changed

  • Made Path::from_bytes_with_nul_unchecked const.
  • Replaced LOOKAHEADWORDS_SIZE (measured in multiples of four bytes) with
    LOOKAHEAD_SIZE (measured in multiples of eight bytes) in driver::Storage
    so that all possible values are valid. (See the lookahead size fix below for
    context.)
  • Require &mut self in driver::Storage::read for compatibility with
    embedded_storage.

Fixed

  • Fixed the lookahead size reported to littlefs2-sys. Previously, the
    reported size was too large by the factor of 8, potentially leading to a
    buffer overflow causing filesystem corruption. Fixing this means that
    Storage::LOOKAHEADWORD_SIZE values that are not a multiple of 2 can now
    lead to an error. Fixes #16.
  • Propagate errors in littlefs callbacks instead of panicking.

Please also create these missing tags:

@nickray
Copy link
Member

nickray commented Feb 7, 2023

I'd add that we expect bumping 0.3->0.4 to not break existing data (and optionally be more clear about which are the two semver-breaking changes requiring the bump).

If there is an unexpected data break, we'd try and fix with a 0.4.1 and yank 0.4.0, correct?

@robin-nitrokey
Copy link
Member Author

Updated.

If there is an unexpected data break, we'd try and fix with a 0.4.1 and yank 0.4.0, correct?

Yes, that would make sense to me.

@@ -1,7 +1,7 @@
[package]
name = "littlefs2"
description = "Idiomatic Rust API for littlefs"
version = "0.3.2"
version = "0.4.0"
authors = ["Nicolas Stalder <[email protected]>", "Brandon Edens <[email protected]>"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add "The Trussed developers"

@nickray nickray merged commit 6b02d54 into trussed-dev:main Feb 7, 2023
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.

LittleFS corruption problem
3 participants