google/benchmark

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

Open

#1.512 aberto em 16 de nov. de 2022

Ver no GitHub
 (3 comments) (1 reaction) (0 assignees)C++ (1.539 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (7.968 stars)
Métricas de merge de PR
 (Mesclagem média 4d 2h) (19 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Modifique a regra cc library no arquivo BUILD para usar o atributo 'includes' em vez de 'strip include prefix' para tratar os cabeçalhos como cabeçalhos de sistema, suprimindo avisos para o código de benchmark.
Pilha de tecnologia
cpp
Domain
build system
Tipo Issue
Funcionalidade
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
C++Bazel
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70