envoyproxy/envoy

Specify -Wdeprecated-declarations for proto targets

Open

#14.281 aberto em 4 de dez. de 2020

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/buildhelp wantedtech debt

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

The deprecated proto fields are generating a whole lot of warnings:

bazel-out/k8-fastbuild/bin/external/envoy_api/envoy/config/core/v3/config_source.pb.h:102:3: note: declared here
  102 |   ApiConfigSource_ApiType_hidden_envoy_deprecated_UNSUPPORTED_REST_LEGACY PROTOBUF_DEPRECATED_ENUM = 0,

We should set the -Wdeprecated-declarations for proto targets to suppress this warning.

Guia do colaborador

Direção de pesquisa
Localize o arquivo BUILD para os alvos proto no repositório e adicione ' Wdeprecated declarations' ao atributo copts.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Bazel
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90