ninja-build/ninja

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

Open

#1.336 geöffnet am 28. Sept. 2017

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.544 Forks)batch import
bugfrontendhelp wanted

Repository-Metriken

Stars
 (10.194 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 39T 3h) (10 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie den Code, der während der Build Ausführung Schrittnummern ausgibt (wahrscheinlich in der Statusanzeigelogik von Ninja), und beheben Sie den Off by One Fehler, bei dem der Schritt 'Installing files' die vorherige Schrittnummer dupliziert, anstatt sie zu erhöhen.
Tech Stack
cpp
Domain
build system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65