Prhub

#25182 chore: add vLLM SPDX copyright headers to ported files

原始 PR 作者 sglang-bot 合并时间 2026-05-14 06:17 文件变更 136 提交数 2 评论 1 代码增减 +255 / -0

执行摘要

为 vLLM 移植文件添加 SPDX 版权头

PR body 指出:已有的 "Adapted from" 注释已体现归属意图,但缺少形式化的 SPDX 标识和上游贡献者版权行,这可能导致许可合规缺口。本次变更机械地补全了这些元数据。

该 PR 属于纯合规性变更,无技术决策需要重点审视。建议快速合并,以扫清许可合规风险。

讨论亮点

唯一的 review 来自 gemini-code-assist[bot],表示没有反馈意见。没有人工 reviewer 提出异议或问题。

实现拆解

  1. 审计识别:通过 grep 筛选出所有包含 "Adapted from .*vllm-project/vllm" 的 Python 文件,共 150 个,其中 14 个已有 SPDX 头,136 个缺少。
  2. 分类处理:对于已有 SPDX-License-Identifier 的文件,仅在其下方插入 SPDX-FileCopyrightText 行;对于完全无 SPDX 的文件,在文件最顶部插入两行(许可证标识 + 版权声明)。
  3. 保留所有现有版权注释:原有的 SGLang Team、HuggingFace、NVIDIA 等版权块以及 "Adapted from" 注释原样保留,仅在上方或下方插入新行。
  4. 特殊处理:例如 mxfp4.py 保留原有 NVIDIA 版权,vLLM 的 SPDX-FileCopyrightText 作为第二条目添加;CRLF 行尾的文件(如 apertus.py)保持原格式。
  5. 验证:PR body 提供验证命令,确保所有标注文件都已覆盖。
文件 模块 状态 重要度
python/sglang/srt/models/baichuan.py 模型层 modified 4.31
python/sglang/srt/models/commandr.py 模型层 modified 4.31
python/sglang/srt/models/dbrx.py 模型层 modified 4.23
python/sglang/srt/models/gemma.py 模型层 modified 4.23

关键源码片段

python/sglang/srt/models/baichuan.py license-header

代表性文件,展示了完整的 SPDX 头添加模式:原有 EleutherAI/HuggingFace 版权块,新头插在文件最顶部。

# SPDX-License-Identifier: Apache-2.0
# SPDX-FileCopyrightText: Copyright contributors to the vLLM project
# Adapted from https://github.com/vllm-project/vllm/blob/main/vllm/model_executor/models/baichuan.py# coding=utf-8
# Copyright 2022 EleutherAI and the HuggingFace Inc. team. All rights reserved.
# ... ( 保留原有版权和许可证 )
"""Inference-only BaiChuan model compatible with HuggingFace weights."""

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

变更仅涉及注释行,不修改任何可执行代码,因此运行时行为零风险。主要风险在于大规模文件修改(136 个文件)可能导致与其他分支的合并冲突,但冲突容易解决。此外,如果未来有文件被移动或重命名,头信息可能需要同步更新。

对用户无功能影响;对项目而言,补齐了许可合规要求,提高了开源合规性。对维护者来说,之后的代码审查不再需要单独检查 SPDX 头,因为本次已统一处理。

批量文件修改可能引起合并冲突 无行为变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论