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

Conflict with serde_yaml #39

Closed
sapsan4eg opened this issue Jul 28, 2017 · 3 comments
Closed

Conflict with serde_yaml #39

sapsan4eg opened this issue Jul 28, 2017 · 3 comments
Labels

Comments

@sapsan4eg
Copy link

Hi, i have issue with using together serde_yaml and config. When I compile an application, I get an error

 0 => yaml::Yaml::Hash(BTreeMap::new()),
   |                               ^^^^^^^^^^^^^^^ expected struct `linked_hash_map::LinkedHashMap`, found struct `std::collections::BTreeMap`

The reason is that the serde_yaml uses the feature "linked-hash-map" of yaml-rust. This feature indicates that type yaml::Hash is equivalent to ::linked_hash_map::LinkedHashMap<Yaml, Yaml>;
https://github.com/chyh1990/yaml-rust/blob/0.3.5/src/yaml.rs#L65

You can reproduce this issue. Just add to Cargo.toml in another project this two crates. And try to compile.

@mehcode
Copy link
Owner

mehcode commented Jul 28, 2017

@sapsan4eg Thanks for reporting. This should be fixed with 71f4b18. Can you try using master and see if your problem is resolved?

[dependencies]
config = { git = "https://github.com/mehcode/config-rs"  } 

@mehcode mehcode added the bug label Jul 28, 2017
@sapsan4eg
Copy link
Author

Yes, it works. Thank you. When is the release expected?

@mehcode
Copy link
Owner

mehcode commented Jul 28, 2017

@sapsan4eg I have tomorrow planned to clean up a few more issues and push a release. I want to take another crack at properly fixing #36 (but still supporting the chain API) before releasing.

@mehcode mehcode closed this as completed Jul 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants