search

georust / geozero

Zero-Copy reading and writing of geospatial data.
Apache License 2.0
336 stars 33 forks source link

Use `prost-build` only with `with-mvt` flag in build script #45

Closed landreussi closed 2 years ago

landreussi commented 2 years ago

This PR makes the lib use prost-build only when the feature with-mvt is enabled, once protobuf stuff is used only in this feature.

I am opening this PR because I am working on a project that doesn't use the with-mvt flag and protobufs and it spends 31s compiling this dep anyway.

I forked this repository, applied this patch, and used it in my project and it worked flawlessly.

I've run the tests and this patch doesn't break anything, but if there is something weird in this PR, please let me know :)

pka commented 2 years ago

Great addition, thank you!