google/benchmark

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

Open

#1 512 ouverte le 16 nov. 2022

Voir sur GitHub
 (3 commentaires) (1 réaction) (0 assignés)C++ (7 968 stars) (1 539 forks)batch import
enhancementhelp wanted

Description

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.

Guide contributeur

Stack technique
cpp
Domaine
build system
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
BazelC++ build
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
Look at the BUILD files in the repository, likely under `src/` or `test/`, to find where `cc library` is used with `strip include prefix`. Examine the googletest repository referenced in the issue to see how they use `includes` instead. Then modify the corresponding `cc library` rules to replace `strip include prefix` with `includes` to mark headers as system headers.