build.rs fails, when the shaderc-rs package from crates.io is used (not a git checkout) about shaderc-rs HOT 8 CLOSED

Workaround: tell cargo to checkout the repository like this:

[build-dependencies]
shaderc = { version = "0.3.1", git="https://github.com/google/shaderc-rs", tag="v0.3.1" }

from shaderc-rs.

I think the solution to this is to remove the git submodule updates from the build script. Other crates that use submodules don't do submodule updates in build scripts - they rely on the user to do that as and when necessary (typically doing git clone --recursive when initially checking out the repository then manual submodule updates as and when necessary).

from shaderc-rs.

Thanks for reporting!

I've noticed the error on docs.rs, and thought it's because not issuing git submodule in the project root directory. So pushed out v0.3.1. It seems the problem persists.

Previously it was OK when using git clone for each project manually. May revert back to that way.

I still like to avoid to require the user to do an additional git clone or git submodule to checkout the projects by themselves and would like to hide everything behind build.rs.

@dylanede: If manual check out is required for a crate, you cannot easily use that crate as an dependency for another project?

from shaderc-rs.

Oh, wait, it seems cargo will automatically checkout submodules for crates on crates.io having .gitsubmodules?

from shaderc-rs.

I've #16 up. Hopefully it will solve the problem.

from shaderc-rs.

With crates in the crates.io registry, Cargo doesn't use git at all. The crates are just a collection of source files kept in the registry that match what you tell cargo publish to upload (typically the entire contents of the crate directory excluding target and git-ignored files, but not necessarily). So if you publish shaderc-rs with the git submodules in a checked out, updated state, their contents will be in the published crate.

What is affected by git is when the crate is specified as a dependency using git = in a Cargo.toml. Off the top of my head I can't remember whether cargo does recursive cloning when using a git-specified dependency. I think it does.

from shaderc-rs.

@dylanede: Ah, that makes sense! Thanks for the explanation!

I checked the .cargo\registry\src\github.com-XXX\shaderc-XXX directory, shaderc and its dependencies are there, and no .git subdirectories in all of them. So that explains the error.

from shaderc-rs.

@HellButcher: v0.3.2 should have fixed the problem. Thanks! :)

from shaderc-rs.