Skip to content

Conversation

@budziq
Copy link
Contributor

@budziq budziq commented Jun 7, 2017

I've added the bitflags example but unfortunately there is a linking error due to other book dependencies using different bitflags versions.

clap -> bitflags 0.8.2
png -> bitflags 0.7.0
openssl -> bitflags 0.9.1

This results in standard problem

error[E0465]: multiple rlib candidates for `bitflags` found
 --> /tmp/rust-skeptic.wsSHTSHfCBhF/test.rs:3:1
  |
3 | extern crate bitflags;
  | ^^^^^^^^^^^^^^^^^^^^^^
...

It the moment I have no idea how to work around such problem in general without changes to skeptic or cargo itself

fixes: https://github.com/brson/rust-cookbook/issues/189
fixes: https://github.com/brson/rust-cookbook/issues/114


Update: image-png and image have released a new version compatible with bitflags "^0.9". Still waiting for clap-rs dependencies updated on crates.io
Update: both image-png, image and clap-rs updated on crates.io but now we are still blocked on pulldown-cmark (skeptic transitive dependency).

@budziq budziq changed the title [blocked] Added "Define and operate on a type represented as a bitfield" example Added "Define and operate on a type represented as a bitfield" example Jun 13, 2017
@brson
Copy link
Contributor

brson commented Jun 15, 2017

Seems like the root bug in skeptic needs to be fixed https://github.com/brson/rust-skeptic/issues/18. This is a horrible problem.

@budziq budziq force-pushed the bitflags branch 3 times, most recently from 23e634f to bbc19cf Compare June 21, 2017 12:35
@budziq
Copy link
Contributor Author

budziq commented Jun 21, 2017

The https://github.com/brson/rust-skeptic/issues/18 will have to be solved otherwise contribution to and maintenance of cookbook will not be realistic (see this issue history).

also updated dependencies to avoid skeptic test buildbreak
@budziq budziq merged commit a0fe9d1 into rust-lang-nursery:master Jul 11, 2017
@budziq budziq deleted the bitflags branch July 12, 2017 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants