Skip to content

chore: update version to 6.5.32#521

Merged
lzwind merged 1 commit intolinuxdeepin:masterfrom
lzwind:master
Apr 16, 2026
Merged

chore: update version to 6.5.32#521
lzwind merged 1 commit intolinuxdeepin:masterfrom
lzwind:master

Conversation

@lzwind
Copy link
Copy Markdown
Contributor

@lzwind lzwind commented Apr 16, 2026

  • bump version to 6.5.32

Log : bump version to 6.5.32

- bump version to 6.5.32

Log : bump version to 6.5.32
@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 6.5.32
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wyu71

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

@lzwind lzwind merged commit 9b92777 into linuxdeepin:master Apr 16, 2026
16 of 17 checks passed
@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

这段代码是 Debian 软件包的 changelog 文件变更记录。从版本控制和软件打包的角度来看,以下是详细的审查意见和改进建议:

1. 语法与格式审查

  • 格式规范:该 diff 符合 Debian changelog 的标准格式(遵循 dpkg-parsechangelog 规范),包括版本号、发行版、紧急程度、变更条目以及维护者信息和时间戳。格式上没有语法错误。

2. 逻辑与内容审查

  • 时间戳异常(严重问题)

    • 问题:维护者日期显示为 Thu, 16 Apr 2026
    • 分析:这是一个未来的日期。虽然 Git 允许提交未来的日期,但在软件发布和构建系统中,未来的时间戳可能会导致严重的逻辑问题。例如,它可能会破坏依赖该包的其他软件的构建顺序,或者导致包管理器(如 APT)在更新时产生混淆,认为该包是"来自未来"的版本。
    • 建议:请务必将日期修正为当前的实际提交日期。
  • 变更描述清晰度

    • chore: add quotes to rm command in postrm script:描述清晰,指出了具体的修改位置(postrm 脚本)和内容(给 rm 命令加引号)。这通常是为了处理文件名中包含空格的情况,是一个好的安全加固实践。
    • chore: update workflows:描述比较笼统。
    • 建议:如果 update workflows 涉及到 CI/CD 流程的关键变更(例如增加构建步骤、修改发布逻辑),建议在 changelog 中稍微具体一点,或者确保相关的 .github/workflows 或构建脚本有详细的提交记录。如果只是微小的格式调整,目前的描述是可以接受的。

3. 代码质量与性能

  • 作为 Changelog 文件,不涉及运行时代码,因此不涉及运行时性能问题。
  • 可维护性:使用了 chore: 前缀,符合常见的 Commit Message 规范(如 Conventional Commits),有助于自动化工具解析和人工阅读,质量良好。

4. 代码安全

  • postrm 脚本的安全性
    • changelog 中提到了 add quotes to rm command。这是一个非常重要的安全修复。
    • 背景:在 Shell 脚本中,如果变量(如 $FILE)未加引号包裹,当文件名包含空格或通配符时,rm $FILE 可能会删除错误的文件或导致脚本报错退出。
    • 评价:这是一个积极的安全改进,能够防止潜在的文件删除错误或路径遍历风险。请确保在 debian/postrm 文件中确实实施了这一修改(例如将 rm $VAR 改为 rm "$VAR")。

总结建议

  1. 修正日期:请立即将时间戳从 2026 年修改为正确的年份(如 20232024),这是当前最需要修正的错误。
  2. 验证脚本:确认 debian/postrm 脚本中所有涉及文件路径操作的变量都已正确加上了双引号。

修正后的示例(假设当前是 2024 年):

+deepin-terminal (6.5.32) unstable; urgency=medium
+
+  * chore: add quotes to rm command in postrm script
+  * chore: update workflows
+
+ -- Liu Zheng <liuzheng@uniontech.com>  Thu, 16 Apr 2024 19:47:34 +0800

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot bot commented Apr 16, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.32
  • Tag SHA: ec6df95c60cea6e956c2e11dd5e8fff3eb1b90f5
  • Commit SHA: d54242ab468f114e714b1cddb0281c718505ddd5
  • Tag Message:
    Release deepin-terminal 6.5.32
    
    
  • Tagger:
    • Name: lzwind
  • Distribution: unstable

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