Skip to content

chore(license): add missing exportPot.bat to REUSE dep5 file declaration#120

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:release/eaglefrom
pppanghu77:release/eagle
May 12, 2026
Merged

chore(license): add missing exportPot.bat to REUSE dep5 file declaration#120
deepin-bot[bot] merged 1 commit into
linuxdeepin:release/eaglefrom
pppanghu77:release/eagle

Conversation

@pppanghu77
Copy link
Copy Markdown

  • Add src/tools/exportPot.bat to the dep5 file list to fix REUSE compliance check failure with reuse >= 6.0

构建(license): 补充 dep5 中遗漏的 exportPot.bat 文件声明

  • 在 dep5 文件列表中补充 src/tools/exportPot.bat,修复 reuse 6.0+ 版本合规检查失败的问题

Log: 补充 dep5 声明文件中遗漏的 exportPot.bat,修复新版本 reuse 工具的开源合规检查失败

- Add src/tools/exportPot.bat to the dep5 file list to fix REUSE compliance check failure with reuse >= 6.0

构建(license): 补充 dep5 中遗漏的 exportPot.bat 文件声明

- 在 dep5 文件列表中补充 src/tools/exportPot.bat,修复 reuse 6.0+ 版本合规检查失败的问题

Log: 补充 dep5 声明文件中遗漏的 exportPot.bat,修复新版本 reuse 工具的开源合规检查失败
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @pppanghu77, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@github-actions
Copy link
Copy Markdown

  • 检测到敏感词export变动
详情
    {
    "export": {
        ".reuse/dep5": {
            "a": [
                "Files: src/tools/exportPot src/tools/importPo src/tools/importPo.bat"
            ],
            "b": [
                "Files: src/tools/exportPot src/tools/exportPot.bat src/tools/importPo src/tools/importPo.bat"
            ]
        }
    }
}

@github-actions github-actions Bot requested a review from liujianqiang-niu May 12, 2026 05:39
@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

你好!我是智能编程助手 CodeGeeX。很高兴为你审查这段代码变更。

这段 git diff 修改的是 .reuse/dep5 文件,这是 REUSE 软件 用来声明版权和许可信息的配置文件。变更的主要内容是在 src/tools 目录的排除/声明列表中,新增了 src/tools/exportPot.bat 文件。

以下是我的详细审查意见:

1. 语法逻辑

  • 正确性:修改逻辑是正确的。原文件只声明了 src/tools/exportPot(通常是无后缀的 Linux/macOS 可执行脚本)和 src/tools/importPo 及其 .bat 后缀的 Windows 批处理文件,但遗漏了 src/tools/exportPot.bat。此次补全确保了 exportPot 在 Windows 下的批处理文件也被正确覆盖,逻辑上没有问题。
  • 格式规范:REUSE 的 dep5 文件格式遵循 Debian 版权文件格式(DEP-5)。Files 字段使用空格分隔多个文件路径,你的修改符合该语法规范。

2. 代码质量

  • 一致性与严谨性:这是一个高质量的修改。它修复了原有配置中的遗漏,使得项目在跨平台(Linux/macOS 与 Windows)的版权和许可证声明上保持了一致性。如果不添加此项,在运行 reuse lint 时,src/tools/exportPot.bat 可能会被报告为缺失版权和许可证信息的文件。

3. 代码性能

  • 无影响.reuse/dep5 是一个纯配置文件,仅在执行 REUSE 规范检查(如 reuse lintreuse spdx)时被解析。增加一个文件路径对解析性能的影响微乎其微,可以忽略不计。

4. 代码安全

  • 潜在风险提示:虽然配置文件本身不涉及执行逻辑,但需要确认声明内容的合理性
    • 该配置将 src/tools/exportPot.bat 的版权分配给了 UnionTech Software Technology Co., Ltd.,并声明其许可证为 CC0-1.0(即无版权限制,公共领域贡献)。
    • 安全审查建议.bat 文件是 Windows 环境下的可执行脚本。将可执行脚本的许可证设置为 CC0-1.0 意味着任何人都可以无限制地使用、修改和分发该脚本。请确认这是项目组的真实意图。通常,项目构建/工具脚本可能会采用与项目主代码相同的许可证(如 GPL-3.0 或 MIT),或者如果它们是非常通用的系统调用,CC0 也是合理的。请确保该 .bat 文件内没有包含敏感信息或专有逻辑。

💡 改进建议

  1. 验证文件存在性:请确保 src/tools/exportPot.bat 文件确实存在于代码库中。如果该文件已被删除或重命名,此条声明将变成死代码(虽然 REUSE 不会报错,但会造成配置冗余)。
  2. 验证文件内部头部注释:REUSE 规范推荐双管齐下。既然在 dep5 中声明了该文件,建议检查 src/tools/exportPot.bat 文件内部是否也添加了对应的 SPDX 标识符(如 SPDX-FileCopyrightTextSPDX-License-Identifier)。虽然 dep5 已经覆盖了声明,但文件内直接注释是更佳的 REUSE 实践。
  3. 运行 Lint 检查:在合并此更改后,建议运行一次 reuse lint,确认该文件不再报缺失许可的警告,且没有引入新的错误。

总结:这是一个正确且必要的补全修改,主要为了完善开源合规性声明。没有语法或逻辑错误,只需确认许可证选择(CC0-1.0)对可执行脚本是否合适即可。

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: max-lvs, pppanghu77

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pppanghu77
Copy link
Copy Markdown
Author

/forcemerge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented May 12, 2026

This pr force merged! (status: unstable)

@deepin-bot deepin-bot Bot merged commit 6d3bc95 into linuxdeepin:release/eagle May 12, 2026
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants