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

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

Open
frederikhors opened this issue Apr 1, 2020 · 7 comments

Comments

@frederikhors
Copy link

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
Copy link
Author

diesel-rs/diesel#2348

@deg4uss3r
Copy link
Collaborator

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 :)

@frederikhors
Copy link
Author

I'm new to Rust.

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

@deg4uss3r
Copy link
Collaborator

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
Copy link
Collaborator

I have a fix for this in deg4uss3r@97a2e29 it is working locally for me :)

@tanriol
Copy link

tanriol commented May 21, 2020

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

@orhun
Copy link

orhun commented Apr 28, 2021

Any updates on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants