-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Rustpkg can't install dependencies that use the src/foo/lib.rs structure #10253
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
Comments
Some packages like https://github.com/brson/rust-sdl fail to work even with |
I encountered a similar (possibly same, at least the same assertion) using |
I have a fix for this, well, sort of -- at least it fixes the ICE. I'm just writing a test for it now. |
Does rustpkg require Github repos to have the same name as the crate inside? For example, can my rust-papi repo define a papi crate, or does the crate have to be called rust-papi? |
Don't pass extra generic arguments in `needless_borrow` fixes rust-lang#10253 Also switches to using `implements_trait` which does ICE when clippy's debug assertions are enabled. changelog: None
If I take a project like https://github.com/steveklabnik/hello, I can manually install via rustpkg just fine:
But, if I have a project that depends on
hello
:It can't properly pull in the hello dependency:
The full debug output dump is here: http://sprunge.us/SRPU but the gist of it is that it has
start_dir = /home/sfackler/test/.rust/build/x86_64-unknown-linux-gnu/src/github.com/steveklabnik/hello-0.1
andJustOne: p = lib.rs
, which is missing thesrc/hello
indirection.A package like https://github.com/sfackler/rust-postgres that has
lib.rs
in the root directory works just fine.cc @catamorphism
The text was updated successfully, but these errors were encountered: