Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

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

Ouverte Adaptée aux débutants
#62 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
70/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
php
Domaine
cli

Piste de recherche

Commence dans src/Archive/Factory.php, où l'extension .phar est associée à PharArchive vers la ligne 95, puis lis PharArchive pour voir comment il appelle PharData. Dans src/GetBinaryCommand.php, installConfig() vers la ligne 216 affiche l'erreur de Generator::generate() mais atteint quand même file_put_contents ; MakeConfigCommand montre le retour FAILURE à reprendre. C'est terminé quand .phar n'atteint plus PharData et qu'une génération échouée n'écrit aucun .rr.yaml. Demande si #49 supprimera ce code avant de commencer.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
PHP
Étoiles
30
Forks
15
Merge moyen
45 min
PR mergées (30 j)
11

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de roadrunner-php/cli

Toutes les issues de roadrunner-php/cli

Issues similaires

Plus d'issues PHP

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.