get-binary exits with 0 when the existing binary is kept
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
从 GetBinaryCommand::execute() 和 installBinary() 开始，这是 issue 中提到的两处。execute() 只有在 installBinary() 返回 null 时才返回 1，但被跳过的文件也会被返回。如果 #66 中引用的验收测试 ExistingBinaryTest::failsWhenBinaryIsKept 存在，请检查它。当保留已有的二进制文件时以 1 退出并输出 RoadRunner has not been installed 警告，即为完成。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- PHP
- 星标
- 30
- 派生
- 15
- 平均合并
- 9 小时 37 分钟
- 30 天内合并 PR
- 16
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
roadrunner-php/cli 的其他 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 72/100
roadrunner-php/cli#67 ·
维护者通常 1 天内回复
查看 roadrunner-php/cli 的全部 Issue
相似的 Issue
-
HttpRequestTrait fatals on WordPress 7.2: $blocked_http_requests clashes with WP_UnitTestCase_Base未关闭
难度 2/5 1-3 小时 新手友好度 72/100
-
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 1/5 1 小时以内 新手友好度 82/100
-
难度 2/5 1-3 小时 新手友好度 65/100
-
难度 2/5 1-3 小时 新手友好度 78/100
pluginsGLPI/addressing#127 ·