build: skip invalid manifests

[FlorianFranzen]

This patch makes sure that only crates that pass some rudimentary check are unpacked in git-deps. This avoids git repos with random and/or broken Cargo.toml files to break your whole build.

The proper fix would be to unpack needed dependencies, but that is a bit more involved, so this should offer a quick and simple fix for now.

[jonringer]

Wouldn't it be preferable to just fix the toml file?

This avoids git repos with random and/or broken Cargo.toml files to break your whole build.

Nevermind, just re-read this. Makes sense.

[FlorianFranzen]

@jonringer I patched this even more by only installing the dependencies request (see here) in which case this would be no longer needed, but I thought this is a good quick fix.

[Patryk27]

Hi, thanks for this merge request; as far as I understand, #167 already solves it, right? (unless there's some additional depth to cargo verify-project that I don't see 🙂)

So in the effort of cleaning-up issues in this repository, I'll just go ahead and close this merge request - but if you feel we could use cargo verify-project, please feel free to re-open it 🙂

Thanks again!