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

sort: disable clippy::suspicious_open_options on OpenBSD #6292

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

lcheylus
Copy link
Contributor

Fix #6290

Lint with clippy and build OK on OpenBSD stable/7.5 and FreeBSD 14.0 (Rust version 1.77).

- Avoid error on OpenBSD stable/7.5 with clippy (lint)
- suspicious_open_options added in Rust 1.77.0 (1.76 used on OpenBSD 7.5)
  https://rust-lang.github.io/rust-clippy/master/index.html#/suspicious_open_options

Fix uutils#6290

Signed-off-by: Laurent Cheylus <[email protected]>
@cakebaker cakebaker merged commit b8aa4e1 into uutils:main Apr 30, 2024
67 of 68 checks passed
@cakebaker
Copy link
Contributor

Thanks!

@tertsdiepraam
Copy link
Member

tertsdiepraam commented Apr 30, 2024

@cakebaker did you see my comment here: #6290 (comment)? Happy to accept this if you want to have it merged, but I'd like to discuss it at least 😄

Edit: Actually, given that this is for CI and we want to make this a tier 1 platform. I agree with this change after all. Some discussion would have been nice though.

@cakebaker
Copy link
Contributor

@tertsdiepraam sorry about that :| Yes, I saw your comment (and their answer), and thought that no further discussion was necessary...

@tertsdiepraam
Copy link
Member

Alright, at least we agree on the outcome 😄

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.

sort: unknown lint clippy::suspicious_open_options on OpenBSD
3 participants