Prhub

#43377 [BugFix] Fix setuptools-rust dep in requirements files

原始 PR 作者 njhill 合并时间 2026-05-22 15:25 文件变更 3 提交数 2 评论 5 代码增减 +2 / -1

执行摘要

修复 setuptools-rust 依赖在多个平台文件中错放与重复

在 ROCm、TPU 平台构建时出现 ModuleNotFoundError: No module named 'setuptools_rust' 错误(见 Issue 评论),XPU 文件中有重复条目。

此 PR 变更简单且正确,但建议在后续 PR 中一并处理 review 指出的其他缺失构建依赖(ninja 等),以确保构建环境完整。

讨论亮点

自动化 review 工具(gemini-code-assist[bot])指出 ROCm build 文件中还缺少 ninja、TPU build 缺少 packagingjinja2、XPU 缺少 ninja,但这些未在 commit 中修复。审核人 Isotr0py 批准了 PR。

实现拆解

  1. ROCm - requirements/build/rocm.txt:添加缺失的 setuptools-rust>=1.9.0 到 build 依赖块。
  2. TPU - requirements/build/tpu.txt:添加缺失的 setuptools-rust>=1.9.0 到 build 依赖块。
  3. XPU - requirements/xpu.txt:删除重复的 setuptools-rust>=1.9.0 条目,仅保留正确位置的一份。
文件 模块 状态 重要度
requirements/build/rocm.txt 构建配置 modified 2.34
requirements/build/tpu.txt 构建配置 modified 2.34
requirements/xpu.txt 依赖配置 modified 1.98

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

缺失其他构建依赖 正确性

自动化 review 指出 ROCm 缺少 ninja、TPU 缺少 packaging 和 jinja2、XPU 缺少 ninja。

结论:未在本次 PR 中处理,但 PR 本身被批准合并。 · unresolved

风险与影响

低风险。仅修改 requirements 文本文件,不涉及代码逻辑。但自动化工具指出的其他缺失构建依赖(ninja, packaging, jinja2)仍未补全,可能导致未来在特定平台出现构建失败。

直接修复 ROCm、TPU 的 Docker 构建失败,消除 XPU 的重复依赖警告。影响范围为这三个平台的构建流程和用户 pip 安装场景。

缺少其他构建依赖未修复

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论