envoyproxy/envoy

Specify -Wdeprecated-declarations for proto targets

Open

#14.281 aperta il 4 dic 2020

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/buildhelp wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Trova il file BUILD per i target proto nel repository e aggiungi ' Wdeprecated declarations' all'attributo copts.
Tech stack
cpp
Dominio
backend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Bazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90