ninja-build/ninja

"Installing files" step always numbered one too low on stdout (i.e. step [N/N] never shown)

Open

#1.336 aberto em 28 de set. de 2017

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)C++ (1.544 forks)batch import
bugfrontendhelp wanted

Métricas do repositório

Stars
 (10.194 stars)
Métricas de merge de PR
 (Mesclagem média 39d 3h) (10 fundiu PRs em 30d)

Description

I presume this is a ninja issue. If it's somehow meson's fault, I'll reopen it over there.

Using build.ninja generated by meson, I always get output like the following:

[1/5] Generating blah.gresource_c with a custom command.
[2/5] Compiling C object blah.gresource@sta/blah.gresource.c.obj.
[3/5] Linking static target libblah.gresource.a.
[4/5] Linking target blah.exe.
[4/5] Installing files.

That is: Step 5 is never shown, and the "Installing files" step is always numbered one lower than it should be, duplicating the step number of the step above it.

ninja 1.8.2 meson 0.42.1 MSYS2/MINGW64 on Windows

Guia do colaborador

Direção de pesquisa
Investigue o código que imprime os números dos passos durante a execução da construção, provavelmente na lógica de impressão de status do ninja, e corrija o erro de deslocamento onde o passo 'Installing files' duplica o número do passo anterior em vez de incrementá lo.
Pilha de tecnologia
cpp
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65