get-binary exits with 0 when the existing binary is kept
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
- 82/100
Piste de recherche
Commencez par GetBinaryCommand::execute() et installBinary(), que le ticket mentionne : execute() ne renvoie 1 que lorsque installBinary() renvoie null, or le fichier ignoré est également renvoyé. Vérifiez le test d'acceptation ExistingBinaryTest::failsWhenBinaryIsKept référencé dans #66 s'il est présent. C'est terminé lorsque la conservation d'un binaire existant se termine avec le code 1 et affiche l'avertissement RoadRunner has not been installed.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Put any file named
rrinto an empty directory. - Run
rr get --no-config -nthere. - 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.
- Langage dominant
- PHP
- Étoiles
- 30
- Forks
- 15
- Merge moyen
- 9 h 37 min
- PR mergées (30 j)
- 16
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de roadrunner-php/cli
-
--stability=rc is treated as devOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
roadrunner-php/cli#67 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de roadrunner-php/cli
Issues similaires
-
Difficulté 2/5 Une demi-journée Accessibilité débutants 68/100
components-web-app/docs#244 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
nhrrob/nhrrob-secure#22 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
PrestaShopCorp/ps_eventbus#501 · 2 réactions ·
-
category::task maintenance priority::normal
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
e0ipso/schema-forms-php#58 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Automattic/blocks-engine#2866 ·
Les mainteneurs répondent en général sous 1 jour