Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

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

未关闭 适合新手
#62 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
70/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
php
领域
cli

调研方向

从 src/Archive/Factory.php 开始,其中约第 95 行将 .phar 扩展名映射到 PharArchive,然后阅读 PharArchive,看它如何调用 PharData。在 src/GetBinaryCommand.php 中,约第 216 行的 installConfig() 会输出 Generator::generate() 的错误,但仍会执行到 file_put_contents;MakeConfigCommand 展示了需要照搬的 FAILURE 返回值。完成标准是:.phar 不再进入 PharData,且生成失败时不写入 .rr.yaml。开始之前请先确认 #49 是否会删除这段代码。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
PHP
星标
30
派生
15
平均合并
45 分钟
30 天内合并 PR
11

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

roadrunner-php/cli 的其他 Issue

查看 roadrunner-php/cli 的全部 Issue

相似的 Issue

更多 PHP Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。