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

.phar assets cannot be opened, and a failed config generation still writes .rr.yaml

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#62 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
70/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
php
Bereich
cli

Rechercherichtung

Beginne in src/Archive/Factory.php, wo die Dateiendung .phar um Zeile 95 herum mit PharArchive abgeglichen wird, und lies dann PharArchive, um zu sehen, wie es PharData aufruft. In src/GetBinaryCommand.php gibt installConfig() um Zeile 216 den Fehler aus Generator::generate() aus, ruft aber trotzdem file_put_contents auf; MakeConfigCommand zeigt die FAILURE-Rückgabe, die übernommen werden sollte. Fertig ist es, wenn .phar nicht mehr PharData erreicht und eine fehlgeschlagene Generierung keine .rr.yaml schreibt. Frage, ob #49 diesen Code entfernen wird, bevor du anfängst.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Description

Two smaller defects found while covering the package with tests. Neither shows up in normal use today.

  • .phar assets cannot be opened. Archive\Factory maps the .phar extension to PharArchive, which opens the file with PharData. PharData refuses that extension: Cannot create phar '.../rr.phar', file extension (or combination) not recognised or the directory does not exist. RoadRunner does not publish .phar assets, so this matcher can never succeed in practice.
  • .rr.yaml is written even when generation fails. In GetBinaryCommand::installConfig(), the error is printed when Generator::generate() throws, but execution continues to file_put_contents($to, $config) with $config undefined. By reading the code, that would emit an undefined-variable warning and write an empty .rr.yaml. MakeConfigCommand returns FAILURE in the same situation. None of the bundled sections throw today, so this path is not reachable with the built-in plugins.

Additional context

Archive handling and binary installation may be replaced by DLoad in #49, in which case this code goes away.

Vorherrschende Sprache
PHP
Sterne
30
Forks
15
Ø Merge
45 Min.
Gemergte PRs (30 T.)
11

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.