get-binary exits with 0 when the existing binary is kept
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 82/100
Direção de pesquisa
Comece em GetBinaryCommand::execute() e installBinary(), que a issue menciona: execute() só retorna 1 quando installBinary() retorna null, mas o arquivo ignorado também é retornado. Verifique o teste de aceitação ExistingBinaryTest::failsWhenBinaryIsKept referenciado em #66, se ele estiver presente. Está concluído quando manter um binário existente encerra com 1 e exibe o aviso RoadRunner has not been installed.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- PHP
- Estrelas
- 30
- Forks
- 15
- Merge médio
- 9h 37min
- PRs com merge (30d)
- 16
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de roadrunner-php/cli
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
roadrunner-php/cli#67 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de roadrunner-php/cli
Issues semelhantes
-
loginWithCustomTokenExchange() fails when an organization is configuredTalvez já em andamento @awss1i assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
auth0/auth0-PHP#883 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
type/bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 75/100
flarum/framework#5231 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
major-release
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 67/100
Intervention/image#1553 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
omegaup/omegaup#10353 · 1 comentário ·
Mantenedores costumam responder em até 5 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 61/100