Skip to content

Update libsqlite3-sys to 0.34.0 #3888

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

partylikeits1983
Copy link

Does your PR solve an issue?

Yes. This PR resolves a version conflict between sqlx and other dependencies (like rusqlite) caused by incompatible versions of the libsqlite3-sys crate, which links the native SQLite library. Cargo requires only one version of such native-linking crates in the dependency graph.

By updating sqlx to use a compatible libsqlite3-sys version, this conflict is eliminated, allowing both libraries to coexist in the project.

Is this a breaking change?

No. This change only updates internal dependencies to align native library versions without changing any public APIs or behavior.

@cvengler
Copy link

I'm having the same issue. This seems like a good fix to me, although the red CI warnings are a bit unsatisfying.

@partylikeits1983
Copy link
Author

I'm having the same issue. This seems like a good fix to me, although the red CI warnings are a bit unsatisfying.

Yeah, this PR is just to demonstrate that this is a fix for me. Obviously other parts of the code base need to be changed before this could be merged.

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