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++ (1 539 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (7 968 stars)
Métriques de merge PR
 (Merge moyen 4j 2h) (19 PRs mergées en 30 j)

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

Direction de recherche
Modifiez la règle cc library dans le fichier BUILD pour utiliser l'attribut 'includes' au lieu de 'strip include prefix' afin de traiter les en têtes comme des en têtes système, supprimant ainsi les avertissements pour le code de benchmark.
Stack technique
cpp
Domaine
build system
Type d'issue
Fonctionnalité
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.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Bazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70