-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
sqlx-macros
disables default features of syn
but doesn't enable required features
#2418
Comments
Two incompatible versions of
This shouldn't be happening. Can you post the generated |
|
Does |
This is definitely weird cause it's showing the correct version here:
|
Nope. I've tried removing |
It's the features we enable.
To fix, just add We're in the middle of an alpha cycle for 0.7.0, I'll make sure to fix it for that release but I don't have the bandwidth to backport right now. |
No Problem, no need to rush. Anyway,
It still won't compile. |
Same errors? |
Yes. Exactly the same errors. |
Try
|
It compiles. Thanks. By the way. I didn't know there is a |
I'd like to leave this open until I have a fix in for 0.7.0, as a reminder. |
sqlx-macros
disables default features of syn
but doesn't enable required features
Released Still leaving this open until I verify that either |
Heh, it appears I already fixed this as part of my big refactors. I remember some funny compilation errors while working on that but didn't think to check if it affected 0.6 as well: https://github.com/launchbadge/sqlx/blame/main/sqlx-macros-core/Cargo.toml#L57 |
I just experienced this issue on sqlx@0.5; I now used the latest version which is @0.7.1; I think the issue is fixed now. |
… to add the following lines to the Cargo.toml file: [build-dependencies] syn = "1" # temporary fix, launchbadge/sqlx#2418
Bug Description
I've asked about this at Discord. They (or He) say this is a bug because
sqlx-macros
dependencies are wrong.Minimal Reproduction
cargo new bugtest
.sqlx
as a dependency.cargo check
Info
rustc --version
: 1.67, 1.68cargo check
outputThe text was updated successfully, but these errors were encountered: