Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

get-binary exits with 0 when the existing binary is kept

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#69 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
82/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
php
Bereich
cli

Rechercherichtung

Beginne bei GetBinaryCommand::execute() und installBinary(), die das Issue nennt: execute() gibt nur dann 1 zurück, wenn installBinary() null liefert, die übersprungene Datei wird aber ebenfalls zurückgegeben. Prüfe den in #66 erwähnten Akzeptanztest ExistingBinaryTest::failsWhenBinaryIsKept, falls er vorhanden ist. Fertig ist es, wenn das Beibehalten einer vorhandenen Binärdatei mit 1 beendet wird und die Warnung RoadRunner has not been installed ausgibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Description

When the binary already exists and is not overwritten — the overwrite question is answered "no", or the command runs with --no-interaction — rr get keeps the old file (correct) but exits with 0 and prints [OK] Your project is now ready. Scripts cannot tell that nothing was installed.

GetBinaryCommand::execute() returns 1 with RoadRunner has not been installed only when installBinary() returns null, but installBinary() returns the skipped file as well.

Expected: exit code 1 and the RoadRunner has not been installed warning when the existing binary is kept.

Steps to reproduce

  1. Put any file named rr into an empty directory.
  2. Run rr get --no-config -n there.
  3. The file is unchanged, the exit code is 0.

Additional context

Found by the acceptance tests in #66 (ExistingBinaryTest::failsWhenBinaryIsKept). The DLoad-based implementation in #49 already returns 1 in this case.

Vorherrschende Sprache
PHP
Sterne
30
Forks
15
Ø Merge
9 Std. 37 Min.
Gemergte PRs (30 T.)
16

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus roadrunner-php/cli

Alle Issues in roadrunner-php/cli

Ähnliche Issues

Weitere Issues zu PHP

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.