Prevent addition of redundant protobuf imports · envoyproxy/envoy#15589

(2 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import

area/buildhelp wanted

Métricas do repositório

we have a lot of redundant import warnings when generating protos

i have raised a pr to fix this upstream in cncf/xds (https://github.com/cncf/xds/pull/3)

i have previously removed them from the envoy codebase (#15329 ) and have just opened another PR to remove more recently added ones (#15588 )

it would be great if we prevented these from being added

the obvious way would be to turn warnings into errors (see https://github.com/protocolbuffers/protobuf/issues/3980)

the problem with this is that there are a lot of warnings about missing dir paths which i think is harder to fix (https://github.com/bazelbuild/bazel/issues/7157)

Direção de pesquisa: Investigue como diferenciar avisos de importação redundante de avisos de diretório ausente no protobuf. Veja os problemas e PRs referenciados para contexto. Considere modificar o sistema de build (Bazel) para tratar avisos de importação redundante como erros, enquanto permite avisos de diretório ausente.
Pilha de tecnologia: None
Domain: backendinfrastructurebuild system
Tipo Issue: Manutenção
Difficulty: 3
Tempo estimado: Meio dia
Status da atividade: Ativo
Clarity: Claro
Prerequisites: GitBazelprotobuf
Simpatia para novatos: 40