Skip to content

Update itertools and allow itertools 0.10 too. - #2745

Merged
emilio merged 1 commit into
mainfrom
itertools
Feb 4, 2024
Merged

Update itertools and allow itertools 0.10 too.#2745
emilio merged 1 commit into
mainfrom
itertools

Conversation

@emilio

@emilio emilio commented Feb 4, 2024

Copy link
Copy Markdown
Contributor

This can help avoid duplicate dependencies when using bindgen as a library.

This can help avoid duplicate dependencies when using
bindgen as a library.
@bobrik

bobrik commented Sep 30, 2025

Copy link
Copy Markdown

@emilio, this is somewhat annoying as it generates random bouncing when unrelated things are bumped:

diff --git a/Cargo.lock b/Cargo.lock
index 29c1f57f0..ac006628b 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -791,7 +791,7 @@ dependencies = [
  "bitflags 2.9.1",
  "cexpr",
  "clang-sys",
- "itertools 0.13.0",
+ "itertools 0.10.5",
  "log",
  "prettyplease 0.2.35",
  "proc-macro2",
@@ -811,7 +811,7 @@ dependencies = [
  "bitflags 2.9.1",
  "cexpr",
  "clang-sys",
- "itertools 0.13.0",
+ "itertools 0.10.5",
  "log",
  "prettyplease 0.2.35",
  "proc-macro2",
@@ -831,7 +831,7 @@ dependencies = [
  "bitflags 2.9.1",
  "cexpr",
  "clang-sys",
- "itertools 0.13.0",
+ "itertools 0.10.5",
  "proc-macro2",
  "quote",
  "regex",
@@ -5218,12 +5218,11 @@ dependencies = [

Is there a better way to do this? Perhaps the answer is to remove itertools here altogether: #3086.

@emilio

emilio commented Sep 30, 2025

Copy link
Copy Markdown
Contributor Author

@bobrik Isn't that a cargo bug if so? It shouldn't downgrade things even if a transitive dependency has an older version of a library.

@bobrik

bobrik commented Oct 1, 2025

Copy link
Copy Markdown

Sure, it's likely a cargo bug. A quick search finds rust-lang/cargo#14115, rust-lang/cargo#5529, rust-lang/cargo#15513, and a bunch of linked issues, all of which with no sign of much progress. While it would be nice to fix the cargo bug, removing the trigger here seems like the next best thing.

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