protobuf: include imports when generating descriptor and add depfile
This CL adds the include_imports flag to protoc when generating
descriptors. This flag is necessary to support any sort of imports in
proto files as GN does not have support for transitively listing all
the source files in a target and its deps.
Since now include_imports silently looks at other source files other
than the ones listed in the target, this means we need a depfile to
allow ninja to pick up if changes are made in transitive dep source
files. Unfortunately, protoc's depfile generation is broken out of the
box and does not work with ninja.
Add support in the wrapper script to correct this mirror what we do in
Perfetto to achieve the same thing (see )
Reviewed-by: Eric Seckler <email@example.com>
Reviewed-by: Bruce Dawson <firstname.lastname@example.org>
Commit-Queue: Lalit Maganti <email@example.com>
1 file changed