Open jayvdb opened 1 month ago
This is a tricky issue, and I guess improving the error message is pretty much all that can be done here.
For when I publish it to crates.io, I actually do a dirty publish, meaning that the style.css
is generated and included in the package that is uploaded, but that file is never tracked so you wouldn't be able to find or pull it from this repo without other steps.
Of course this could be committed to the repo, but I'm very strongly against committing any file that can be generated on the fly during the build.
Putting the file in the package in this way is just a workaround because the submodule information isn't saved.
But in case of a cargo install --git ...
we might have a chance that we can pull in the submodule from the build.rs
as an extra step, just have to be careful to detect if we are in a git repo or not (to not break the regular cargo install
).
I'll have a go at it and we'll see how it goes (not sure when I get to it though. Currently I'm not making much use of my own tool :sweat_smile:)
I think using install --git
will be rare, so probably not worth the effort at this stage.
Describe the bug
Local build from a new repo fails with the following if the submodules havent been fetched:
Expected behavior
An error explaining git submodules need to be fetched.
Or better, a warning that no themes will be available, so that
cargo install --git https://github.com/dnaka91/llvm-cov-pretty
will work albeit degraded.To reproduce
No response
What operating system are you using?
None
Terminal output
No response
Additional context
No response