Skip to content

Switch serde dependency to serde_core#1285

Merged
dtolnay merged 1 commit intoserde-rs:masterfrom
dtolnay:serdecore
Sep 13, 2025
Merged

Switch serde dependency to serde_core#1285
dtolnay merged 1 commit intoserde-rs:masterfrom
dtolnay:serdecore

Conversation

@dtolnay
Copy link
Copy Markdown
Member

@dtolnay dtolnay commented Sep 13, 2025

No description provided.

Comment thread Cargo.toml
memchr = { version = "2", default-features = false }
ryu = "1.0"
serde = { version = "1.0.194", default-features = false }
serde_core = { version = "1.0.220", default-features = false }
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I tried this as well but it does not work because it collides with the dev-dependency during test builds.

serde = { package = "serde_core", version = "1.0.220", default-features = false }
error[E0464]: multiple candidates for `rlib` dependency `serde` found
  --> src/de.rs:16:5
   |
16 | use serde::de::{self, Expected, Unexpected};
   |     ^^^^^
   |
   = note: candidate #1: /git/serde-json/target/debug/deps/libserde-7f9d001a7d3b68aa.rlib
   = note: candidate #2: /git/serde-json/target/debug/deps/libserde_core-33dd47b23065293c.rlib

@dtolnay dtolnay merged commit 81ba3aa into serde-rs:master Sep 13, 2025
16 checks passed
@dtolnay dtolnay deleted the serdecore branch September 13, 2025 23:31
takumi-earth pushed a commit to earthlings-dev/json that referenced this pull request Jan 27, 2026
Switch serde dependency to serde_core
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant