Skip to content

Try to update dependencies - #1636

Open
Devel08 wants to merge 1 commit into
bytecodealliance:mainfrom
Devel08:main
Open

Devel08 wants to merge 1 commit into
bytecodealliance:mainfrom
Devel08:main

Conversation

@Devel08

@Devel08 Devel08 commented Jun 21, 2026

Copy link
Copy Markdown

this pr updates bitflags, serial_test and criterion. the rustc-dep-of-std flag is no longer part of bitflags.

@xtqqczze

Copy link
Copy Markdown
Contributor

@Devel08 This branch has conflicts that must be resolved

@Devel08

Devel08 commented Oct 15, 2026

Copy link
Copy Markdown
Author

@Devel08 This branch has conflicts that must be resolved

Updated deps again and resolved the conflicts, but I don't think this will be merged anytime soon :/

Comment thread Cargo.toml

[dependencies]
bitflags = { version = "2.9.2", default-features = false }
bitflags = { version = "2.13.0", default-features = false }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this isn't necessary?

Comment thread Cargo.toml
libc = "0.2.182"
libc_errno = { package = "errno", version = "0.3.10", default-features = false }
serial_test = "2.0.0"
serial_test = "4.0.1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

serial_test v4 has a MSRV of 1.93. Although it's probably fine since it's a dev dependency, what is the motivation over say, v3?

Comment thread Cargo.toml

[target.'cfg(all(criterion, not(any(target_os = "emscripten", target_os = "wasi"))))'.dev-dependencies]
criterion = "0.4"
criterion = "0.8.2"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Again criterion v0.8 is a dev dependency with a MSRV of 1.86, what does it gain us though?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Well, I just thought that it's a good practice to keep the dependencies on the latest versions, to keep msrv of 1.65 we can do bitflags 2.13.2, criterion 0.5.1, and serial_test 3.3.1, as they're being compiled succesfully with $ cargo +1.65.0 build. However, if this provides no benefit, we better close that

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.

2 participants