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

v2.7.0 fails to build on rust < 1.77.0 #51

Closed
SRv6d opened this issue Sep 5, 2024 · 4 comments · Fixed by #52
Closed

v2.7.0 fails to build on rust < 1.77.0 #51

SRv6d opened this issue Sep 5, 2024 · 4 comments · Fixed by #52
Assignees

Comments

@SRv6d
Copy link

SRv6d commented Sep 5, 2024

Starting with v2.7.0, codspeed-criterion-compat fails to build on rust versions lower than 1.77.0.

error: unsupported output in build script of codspeed-criterion-compat v2.7.0: cargo::rustc-check-cfg=cfg(codspeed)
Found a cargo::key=value build directive which is reserved for future use.
Either change the directive to cargo:key=value syntax (note the single :) or upgrade your version of Rust.
See https://doc.rust-lang.org/cargo/reference/build-scripts.html#outputs-of-the-build-script for more information about build script outputs.

I assume a dependency is using rust-lang/cargo#12201 but haven't looked further into it. Maybe codspeed could add a fixed MSRV and test it in CI?

@art049 art049 self-assigned this Sep 6, 2024
@art049
Copy link
Member

art049 commented Sep 6, 2024

Yes, this 2.7.0 had to add this to silence linter warnings, hence those things has been added to the build scripts, but you're right, since we don't test with an MSRV I didn't notice the issue upon release.

@SRv6d
Copy link
Author

SRv6d commented Sep 6, 2024

Is there a way we can fix this? Codspeed is great but it is slightly ironic that it is what was causing CI failures for me.
I can pin codspeed to a version that is compatible with my MSRV, but It would obviously be nicer to have a known MSRV. 1.77.0 seems a bit young for a MSRV but this is a decision for codpseed to make.

Copy link
Member

art049 commented Sep 6, 2024

Yes 1.77 would be a bit too young, we'll go with 1.74. And we'll be checking the MSRV in the CI as you mentionned

@art049 art049 closed this as completed in #52 Sep 6, 2024
@art049 art049 changed the title v2.7.0 fails to build on rust > 1.77.0 v2.7.0 fails to build on rust < 1.77.0 Sep 6, 2024
@SRv6d
Copy link
Author

SRv6d commented Sep 6, 2024

Nice, that was fast, almost benchmark worthy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants