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

could not compile sqlparser #1612

Open
tobyhede opened this issue Dec 19, 2024 · 3 comments
Open

could not compile sqlparser #1612

tobyhede opened this issue Dec 19, 2024 · 3 comments

Comments

@tobyhede
Copy link
Contributor

tobyhede commented Dec 19, 2024

With a clean clone of the repo, cargo test results in a compile error.

It might be my local, but tried with Rust 1.75, 1.78 and stable.

error[E0658]: associated type bounds are unstable
    --> src/ast/mod.rs:7876:43
     |
7876 |             I: IntoIterator<Item = usize, IntoIter: DoubleEndedIterator + Clone> + Clone,
     |                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     |
     = note: see issue #52662 <https://github.com/rust-lang/rust/issues/52662> for more information

For more information about this error, try `rustc --explain E0658`.
error: could not compile `sqlparser` (lib test) due to 1 previous error
warning: build failed, waiting for other jobs to finish...
warning: associated type bounds are unstable
    --> src/ast/mod.rs:7876:43
     |
7876 |             I: IntoIterator<Item = usize, IntoIter: DoubleEndedIterator + Clone> + Clone,
     |                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     |
     = note: see issue #52662 <https://github.com/rust-lang/rust/issues/52662> for more information
     = warning: unstable syntax can change at any point in the future, causing a hard error!
     = note: for more information, see issue #65860 <https://github.com/rust-lang/rust/issues/65860>

warning: `sqlparser` (lib) generated 1 warning
@mvzink
Copy link
Contributor

mvzink commented Dec 20, 2024

Compiles for me on 1.79+ (see release notes). If this is the only thing in the way of lowering MSRV, it's just test code and could probably be rewritten as a macro. 1.75 builds & tests pass if this test is removed.

@tobyhede
Copy link
Contributor Author

There is a .tool-versions file that defines 1.75.0.
I am using mise which probably picked that up as the rust to use.

@tobyhede
Copy link
Contributor Author

It might be worth either removing or updating .tool_versions?

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

No branches or pull requests

2 participants