Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Aperta Adatta ai principianti
#62 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
70/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
php
Ambito
cli

Direzione di ricerca

Inizia da src/Archive/Factory.php, dove l'estensione .phar viene associata a PharArchive intorno alla riga 95, poi leggi PharArchive per vedere come chiama PharData. In src/GetBinaryCommand.php, installConfig() intorno alla riga 216 stampa l'errore di Generator::generate() ma raggiunge comunque file_put_contents; MakeConfigCommand mostra il ritorno FAILURE da copiare. È completato quando .phar non raggiunge più PharData e una generazione fallita non scrive alcun .rr.yaml. Chiedi se #49 rimuoverà questo codice prima di iniziare.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
PHP
Stelle
30
Fork
15
Merge medio
45m
PR unite (30g)
11

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di roadrunner-php/cli

Tutte le issue di roadrunner-php/cli

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.