Feature: Generate protos using a user defined prost_build::Config

[SkamDart]

Feature Request

Add the ability to compile protos using a user defined prost_build::Config.

Crates

tonic-build

Motivation

#314 beat me to opening a similar issue. I have a repro here for failing to compile google protobuf types.

prost provides the ability to compile_well_known_types to generate their well known types from the corresponding .proto files. However, the tonic-build crate uses the default constructor

Proposal

Add an additional function, compile_with_builder similar to tonic_build::compile that includes a parameter for a user defined prost_build::Config.

Alternatives

If we want to solve a subset of the problem, we could call the compile_well_known_types for all builds.

FWIW - I'm probably missing or some reason why you didn't do this originally.

[rlabrecque]

Hmm it used to?

#49

[LucioFranco]

@SkamDart everything seems to compile fine if I include prost-types as a dependency? Is there a specific reason that doesn't work for you?

I do think we can include a way for users to manually use the prost config, I'll open a PR for that.

[LucioFranco]

Ok Pr has been merged to support custom prost configs #318

[SkamDart]

@LucioFranco Incredible. Thank you very much! I was going to happily do the dev work myself but you beat me to it.

[SkamDart]

As for why it doesn't compile without that? It's likely that, I'm not including prost-types.

[LucioFranco]

@SkamDart yup, if you look at the error it shoudl say its looking for prost-types but its not included in the cargo.toml.