Skip to content

update to tempfile 3.27.0#563

Merged
Freax13 merged 3 commits into
rust-osdev:mainfrom
Freax13:fix/nightly-2026-05-10
May 14, 2026
Merged

update to tempfile 3.27.0#563
Freax13 merged 3 commits into
rust-osdev:mainfrom
Freax13:fix/nightly-2026-05-10

Conversation

@Freax13

@Freax13 Freax13 commented May 10, 2026

Copy link
Copy Markdown
Member

Previously, tempfile depended on an old rustix version that no longer builds on the latest nightly compiler. Bumping the tempfile crate also bumps rustix.

Supersedes #561

Freax13 added 3 commits May 10, 2026 09:09
Previously, tempfile depended on an old rustix version that no longer
builds on the latest nightly compiler. Bumping the tempfile crate also
bumps rustix.
Something in cargo changed and now cargo clippy also seems to lint
binary dependencies. While that's reasonable, it also cause features to
be unified across the bootloader and test kernel crates and that's
causing issues.
Fix this by excluding the test kernels. The test kernels were already
linted seperatedly anyway.
@Freax13

Freax13 commented May 10, 2026

Copy link
Copy Markdown
Member Author

@jarovo I saw that you also made changes in #561 to move the disk buffer around. I'm not quite sure why that's needed. Is there a reason to do this?

@jarovo

jarovo commented May 10, 2026

Copy link
Copy Markdown

@jarovo I saw that you also made changes in #561 to move the disk buffer around. I'm not quite sure why that's needed. Is there a reason to do this?

Thanks for looking at the code and asking. I've just updated the PR description to specify the reasons for the changes.

@phil-opp phil-opp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thank you!

@Freax13 Freax13 merged commit 5d97b89 into rust-osdev:main May 14, 2026
9 checks passed
@Freax13

Freax13 commented May 14, 2026

Copy link
Copy Markdown
Member Author

I don't think a release is necessary for this, it should be enough for user to just run cargo update. Please let me know if I'm wrong.

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.

3 participants