cargo-outdated icon indicating copy to clipboard operation
cargo-outdated copied to clipboard

warning: Feature uuidv07 of package diesel has been obsolete in version 1.4.4

Open frederikhors opened this issue 5 years ago • 7 comments

I have this in my Cargo.toml:

[dependencies]
actix-web = "=2.0.0"
actix-rt = "=1.0.0"
chrono = { version = "=0.4.11", features = ["serde"] }
diesel = { version = "=1.4.4", features = ["postgres", "r2d2", "uuidv07", "chrono"] }
diesel_migrations = "=1.4.0"
dotenv = "=0.15.0"
env_logger = "=0.7.1"
lazy_static = "=1.4.0"
log = "=0.4.8"
r2d2 = "=0.8.8"
serde = "=1.0.105"
serde_json = "=1.0.50"
uuid = { version = "=0.8.1", features = ["serde", "v4"] }

It works because of uuidv07 feature but cargo outdated gives me this warning:

warning: Feature uuidv07 of package diesel has been obsolete in version 1.4.4.

Why?

frederikhors avatar Apr 01 '20 16:04 frederikhors

https://github.com/diesel-rs/diesel/issues/2348

frederikhors avatar Apr 01 '20 16:04 frederikhors

Hi @frederikhors thanks for this issue!

This is because uuidv07 isn't explicitly in the [features] section of the Cargo.toml but is mapped to the uuid package here:

uuidv07 = { version = "0.7.0", optional = true, package = "uuid"}

I think this is a small UI bug and I am creating a patch for it. I have something working I just need to clean it up :)

deg4uss3r avatar Apr 01 '20 20:04 deg4uss3r

I'm new to Rust.

Is this a problem with diesel also or only with cargo-outdated?

frederikhors avatar Apr 01 '20 21:04 frederikhors

No worries, looks like just cargo-outdated is parsing this slightly wrong.

In the diesel changelog.md; however, it looks like the next release will rename this touuid and drop support for older versions of uuid.

deg4uss3r avatar Apr 01 '20 21:04 deg4uss3r

I have a fix for this in https://github.com/deg4uss3r/cargo-outdated/commit/97a2e2979576e6efe02369b0f4dfec3571402e29 it is working locally for me :)

deg4uss3r avatar Apr 07 '20 14:04 deg4uss3r

Wouldn't it be enough to change in features_and_options the Dependency::package_name to be Dependency::name_in_toml?

tanriol avatar May 21 '20 10:05 tanriol

Any updates on this?

orhun avatar Apr 28 '21 19:04 orhun