Skip to content

chore: update version to 6.5.51#252

Merged
wyu71 merged 1 commit intolinuxdeepin:masterfrom
wyu71:master
Feb 27, 2026
Merged

chore: update version to 6.5.51#252
wyu71 merged 1 commit intolinuxdeepin:masterfrom
wyu71:master

Conversation

@wyu71
Copy link
Contributor

@wyu71 wyu71 commented Feb 27, 2026

  • bump version to 6.5.51

Log : bump version to 6.5.51

- bump version to 6.5.51

Log : bump version to 6.5.51
@github-actions
Copy link

TAG Bot

TAG: 6.5.51
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

deepin pr auto review

这段代码变更主要涉及版本号的更新和 changelog 的维护。以下是我的审查意见:

1. 语法逻辑

  • 语法正确debian/changeloglinglong.yaml 的格式符合各自的规范,语法没有错误。
  • 版本一致性debian/changelog 中声明版本为 6.5.51,而 linglong.yaml 中版本为 6.5.51.1。这通常是预期的行为(Debian 包版本 vs Linglong 包版本),但需要确认项目内部的版本管理策略是否允许这种差异。

2. 代码质量

  • Changelog 规范性
    • 遵循了标准的 Debian changelog 格式。
    • 使用了约定式提交(Conventional Commits)风格(如 fix:, feat:),这有助于自动化工具解析和生成发布说明,质量较高。
  • 描述清晰度
    • "resolve file save failure on SMB filesystem" 和 "resolve DOCX conversion timeout for large files" 描述了具体的修复内容,清晰明了。
    • "Unify linglong.yaml" 稍显模糊,建议补充具体统一了哪些内容(例如:配置项、依赖版本等)。

3. 代码性能

  • 此变更不涉及代码逻辑,因此对性能无直接影响。

4. 代码安全

  • 潜在风险
    • "add USE_PDFIUM_BUNDLE option to support external pdfium (feat: add USE_PDFIUM_BUNDLE option to support external pdfium #247)" 引入了支持外部 pdfium 的选项。这需要确保:
      • 外部 pdfium 的来源可信(避免加载恶意库)。
      • 版本兼容性(外部 pdfium 的 API 是否与当前代码兼容)。
    • 建议在代码中对外部 pdfium 的加载路径进行严格校验,防止路径劫持攻击。

改进建议

  1. Changelog 细化
    • 将 "Unify linglong.yaml" 改为更具体的描述,例如:
      chore: unify dependency versions in linglong.yaml
  2. 版本号管理
    • 如果项目使用自动化版本管理工具(如 gbpdch),建议确保 debian/changeloglinglong.yaml 的版本号同步更新。
  3. 外部库安全
    • 在加载外部 pdfium 时,增加签名校验或哈希校验,确保加载的库未被篡改。
    • 在文档中明确说明 USE_PDFIUM_BUNDLE 的使用场景和安全注意事项。

总结

这段变更整体质量较高,符合规范。建议进一步细化 changelog 描述,并关注外部 pdfium 加载的安全性。

@deepin-ci-robot
Copy link

[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

@wyu71 wyu71 merged commit 005a92e into linuxdeepin:master Feb 27, 2026
7 checks passed
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Feb 27, 2026

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.51
  • Tag SHA: 3e7a8f7d0060ed695d96b788644b728fb46fbff2
  • Commit SHA: 1c370a2b432f6fee5d33ae82febf4240be3780c4
  • Tag Message:
    Release deepin-reader 6.5.51
    
    
  • Tagger:
    • Name: wyu71
  • 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