mesonbuild/meson

compiler.get_supported_arguments reports success for unsupported flags

Open

#5.355 aberto em 3 de mai. de 2019

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Python (1.465 forks)batch import
bugcompilersgood first issue

Métricas do repositório

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

Description

The has_argument and get_supported_argument methods are reporting success when the compiler returns an unsupported flag error to stderr.

I have attached a simple test case which demonstrates the error. The relevant meson.log entry is

Running compile:
Working directory:  /tmp/tmpygkykj04
Command line:  c++ /tmp/tmpygkykj04/testfile.cpp -pipe -D_FILE_OFFSET_BITS=64 -c -o /tmp/tmpygkykj04/output.obj -O0 -fpermissive -Wdeclaration-after-statement 

Code:
 int i;

Compiler stdout:
 
Compiler stderr:
 cc1plus: warning: command line option '-Wdeclaration-after-statement' is valid for C/ObjC but not for C++

Compiler for C++ supports arguments -Wdeclaration-after-statement: YES

meson.build.txt

Guia do colaborador

Direção de pesquisa
Examine como os métodos `has argument` e `get supported argument` processam a saída stderr do compilador e certifique se de que eles verificam mensagens de erro antes de relatar sucesso.
Pilha de tecnologia
python
Domain
build system
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
MesonC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60