Side-effect on `std::convert::Into<_>` introduced in 0.1.12

[behnam]

As the 0.1.12 update yesterday, I noticed that building lib unit tests is failing in unicode-bidi with flame_it feature. I went back and minimized the changes for the break, which actually reduced to only extern crate flame;, and only happening for =0.1.12.

Here's the commit causing error on a stable unicode-bidi build: https://github.com/behnam/rust-unicode-bidi/commit/6e6fc7890bca1e1fe952e69508dba9369e1d37b1

Here's the error:

error[E0283]: type annotations required: cannot resolve `level::Level: std::convert::Into<_>`
   --> src/level.rs:331:31
    |
331 |         assert_eq!(1u8, level.into());
    |                               ^^^^

error: aborting due to previous error

from https://travis-ci.org/behnam/rust-unicode-bidi/jobs/243806493#L259

I have no idea how flame can cause this in a module that doesn't use any of the traits. What do you think, @TyOverby ?

[TyOverby]

Could it be an issue with flame changing transitive dependencies?

[behnam]

Hum... not sure what you mean by transitive dependencies.

I'm looking at git diff 590014d e1cf412 and can't find anything suspicious.

[TyOverby]

@behnam I mean, could adding flame as a dependency change the versions of other dependencies in your project?

[TyOverby]

@behnam: any update on this? I don't think that this is the fault of bincode.

[behnam]

No updates here, @TyOverby . I guess it's a version conflict for serde_derive stuff, but I don't find it.

[TyOverby]

If this still repros for you, could you send me the contents of your Cargo.lock file before and after?

Also try building flame with and without the json feature enabled.

[behnam]

Thanks, @TyOverby. Here are the files:

• before: https://gist.github.com/behnam/fc310681cb5b90f1218966174db7e6ce
• after: https://gist.github.com/behnam/52082c3c920c74ee43ac692a83e85502

[behnam]

Also, I disabled default flame features, but still got the same error.

[TyOverby]

your before/after cargo.lock.toml files are identical

[TyOverby]

Closing due to inactivity; please reopen if this is still an issue.