8000 Latest patch bumped MSRV for logos-derive · Issue #449 · maciejhirsz/logos · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Latest patch bumped MSRV for logos-derive #449

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

Closed
elpiel opened this issue Dec 6, 2024 · 6 comments
Closed

Latest patch bumped MSRV for logos-derive #449

elpiel opened this issue Dec 6, 2024 · 6 comments
Labels
question Further information is requested

Comments

@elpiel
Copy link
elpiel commented Dec 6, 2024

Given that we used protox crate which has set logos to 0.14,since the latest update it now requires MSRV of 1.74.

What is your policy for MSRV bumping?
I see that the latest MSRV in the CHANGELOG was mentioned but not the latest one (for 0.14.3)

@jeertmans
Copy link
Collaborator

Indeed, we bumped the MSRV without documenting it, as we don't have any specific policy for this. I usually don't mind bumping MSRV if that enables new features, but we could at least document that.

@jeertmans jeertmans added the question Further information is requested label Dec 6, 2024
@elpiel
Copy link
Author
elpiel commented Dec 6, 2024

Yes, well protox defines MSRV which is what we use but logos broke that MSRV for protox, so now protox also doesn't have proper MSRV.
I don't mind when crates break builds because of msrv especially if the are only dev-deps, however, it's less than idea to set a third party crates versions to circumvent this though.

@jeertmans
Copy link
Collaborator

Yes, well protox defines MSRV which is what we use but logos broke that MSRV for protox, so now protox also doesn't have proper MSRV.

You mean because of us, or because you never had a proper MSRV?

I don't mind when crates break builds because of msrv especially if the are only dev-deps, however, it's less than idea to set a third party crates versions to circumvent this though.

I am not sure to understand the second part of your sentence. To me, it makes sense that bumping dependencies might require bumping your project's MSRV, no?

@maciejhirsz
Copy link
Owner

MSRV bump is technically a breaking change, so it should probably warrant a 0.15 release I reckon.

@elpiel
Copy link
Author
elpiel commented Dec 7, 2024

You mean because of us, or because you never had a proper MSRV?

I'm saying that we don't use logos directly but it broke our build even though protox has strict Msrv defined.

I am not sure to understand the second part of your sentence. To me, it makes sense that bumping dependencies might require bumping your project's MSRV, no?

Not with path versions.

@maciejhirsz yes, I suppose a bump in the minor version would solve the issues for the future.

@jeertmans
Copy link
Collaborator
jeertmans commented Dec 7, 2024

MSRV bump is technically a breaking change, so it should probably warrant a 0.15 release I reckon.

I agree that it can be considered a breaking change (e.g., dropping support for a Python version in is usually considered a breaking changes), but Rust is also very different as it releases new versions more frequently and update are very easy: all to say that it might be better to define a clear versioning policy.

I'm saying that we don't use logos directly but it broke our build even though protox has strict Msrv defined.

Yeah, I forgot that Rust crates usually specify dependency versions as x.y and not x.y.z, meaning that the z (patch) part can be freely updated.

Not with path versions.

It's true, this is just that I am more used to the standard semantic versioning, where bumping x means breaking changes, bumping y means new features and z means fixes (i.e., patches). As we haven't reached 1.0.0 yet, I wasn't sure of what direction to take regarding versioning.

I will yank 0.14.3 and release it under 0.15, so hopefully this will too many projects breaking.

jeertmans added a commit that referenced this issue Dec 7, 2024
Release 0.14.3 as 0.15, as it bumped the MSRV, see #449.
jeertmans added a commit that referenced this issue Dec 7, 2024
Release 0.14.3 as 0.15, as it bumped the MSRV, see #449.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants
0