get-binary exits with 0 when the existing binary is kept
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
Línea de trabajo
Empieza en GetBinaryCommand::execute() y installBinary(), que el issue menciona: execute() solo devuelve 1 cuando installBinary() devuelve null, pero el archivo omitido también se devuelve. Revisa la prueba de aceptación ExistingBinaryTest::failsWhenBinaryIsKept referenciada en #66 si está presente. Está terminado cuando mantener un binario existente sale con 1 e imprime la advertencia RoadRunner has not been installed.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- PHP
- Estrellas
- 30
- Forks
- 15
- Merge medio
- 45 min
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de roadrunner-php/cli
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
roadrunner-php/cli#68 ·
Los mantenedores suelen responder en 1 día
-
--stability=rc is treated as devAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
roadrunner-php/cli#67 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
roadrunner-php/cli#62 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
roadrunner-php/cli#61 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
roadrunner-php/cli#60 ·
Los mantenedores suelen responder en 1 día
Todos los issues de roadrunner-php/cli
Issues similares
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 4 días
-
Перевод устарел
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
bug
Dificultad 2/5 Medio día Aptitud para principiantes 76/100
m3ue/m3u-editor#1604 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
femiwiki/docker-mediawiki#1497 ·
Los mantenedores suelen responder en 1 día