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

fix: cargo-auditable build error #1929

Merged
merged 1 commit into from
Apr 6, 2025

Conversation

Prince213
Copy link
Contributor

Use futures-lite instead of dep:futures-lite in gix-packetline and gix-protocol.

This currently affects NixOS.

Related:

Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

I think there needs to be a comment around these, otherwise these will seem like an oversight and changed back sometime in the future. These comments should contain links for more context.

#1024 is a decent description of what's going on, and it doesn't look like the reported bug in Cargo is fixed yet.

Use `futures-lite` instead of `dep:futures-lite` in gix-packetline and
gix-protocol.

See-Also: rust-secure-code/cargo-auditable#124
@Prince213 Prince213 force-pushed the fix-cargo-auditable branch from 25b6e3d to 11bd59f Compare April 5, 2025 11:45
@Prince213 Prince213 requested a review from Byron April 5, 2025 12:11
Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@Byron Byron merged commit 19f50d0 into GitoxideLabs:main Apr 6, 2025
21 checks passed
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