google/benchmark

[FR] cc_libraries should use "includes", not "strip_include_prefix"

Open

#1512 aperta il 16 nov 2022

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)C++ (1539 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (7968 star)
Metriche merge PR
 (Merge medio 1g 23h) (21 PR mergiate in 30 g)

Descrizione

So that when users import google-benchmark, the headers are included as "system headers", instead of regular headers.

If this is not done, compilers will throw warnings at google-benchmark code, which users have no way to act upon. Users should be able to choose a highly restrictive set of warnings without having to fix them in google-benchmark.

Note that this pattern is already applied in the googletest repo.

Guida contributor

Direzione di ricerca
Esamina i file BUILD nel repository, probabilmente in `src/` o `test/`, per trovare dove viene utilizzato `strip include prefix` con `cc library`. Controlla il repository googletest menzionato nell'issue per vedere come usano `includes` invece. Quindi modifica le regole `cc library` corrispondenti per sostituire `strip include prefix` con `includes` per contrassegnare gli header come header di sistema.
Tech stack
cpp
Dominio
build system
Tipo issue
Funzionalità
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
BazelC++ build
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80