Consider switching back from toml_edit to basic-toml

[djc]

toml_edit has relatively heavy dependencies, and, in particular, is more aggressive about MSRV bumps.

[bkchr]

@djc would you be open to provide a pr?

[djc]

I can certainly take a look at it if you're open to the idea!

[djc]

Ahh, I forgot that this was not so straightforward. How would you feel about including a minimal, no-dependency parser for the subset of TOML that the crate needs?

[bkchr]

Sounds also possible. I mean you are right that we don't need that much functionality.

[sdbondi]

Somewhat related: You currently cannot use indexmap with the borsh feature and borsh with the derive feature due to toml_edit in this crate

[dependencies]
borsh = {version="*", features = ["derive"]}
indexmap = { version = "2.6.0", features =["borsh"]}

error: cyclic package dependency: package `borsh v1.5.3` depends on itself. Cycle:
package `borsh v1.5.3`
    ... which satisfies dependency `borsh = "^1.2"` of package `indexmap v2.6.0`
    ... which satisfies dependency `indexmap = "^2.3.0"` of package `toml_edit v0.22.22`
    ... which satisfies dependency `toml_edit = "^0.22.20"` of package `proc-macro-crate v3.2.0`
    ... which satisfies dependency `proc-macro-crate = "^3"` of package `borsh-derive v1.5.3`
    ... which satisfies dependency `borsh-derive = "~1.5.3"` of package `borsh v1.5.3`

[zeenix]

So I got totally nerd-snipped by this and I wrote a TOML parsing library with minimum dependencies. Most importantly, it does not depend on indemap. It depends only on winnow and optionally on serde and std. It also provide specific API for Cargo.toml parsing, that this crate can really use.

I'm currently porting the code and will provide a PR soon. I hope it's welcomed. :)