Skip to content

chore: add Pkgconfig in CMake configuration - #27

Merged
lzwind merged 1 commit into
linuxdeepin:masterfrom
add-uos:master
Oct 17, 2025
Merged

chore: add Pkgconfig in CMake configuration#27
lzwind merged 1 commit into
linuxdeepin:masterfrom
add-uos:master

Conversation

@add-uos

@add-uos add-uos commented Oct 17, 2025

Copy link
Copy Markdown
Contributor
  • Add PkgConfig requirement from CMake configuration

Log: Update build dependencies for license compliance

 - Add PkgConfig requirement from CMake configuration

Log: Update build dependencies for license compliance
@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 0.1.17
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

根据提供的git diff,我将对代码进行审查,并提出改进意见:

  1. 版本更新和依赖调整:
  • 版本从0.1.16更新到0.1.17,这是正常的版本迭代
  • 添加了pkg-config作为构建依赖,这个调整是合理的,因为它有助于管理库依赖关系
  1. debian/changelog 分析:
  • changelog格式符合Debian包管理规范
  • 更新记录清晰,包含了版本号、更新类型和作者信息
  • 时间戳设置为2025年10月17日,这是未来的日期,建议修改为当前日期
  1. debian/control 分析:
  • 维护者信息正确
  • 构建依赖关系合理,同时支持Qt5和Qt6版本
  • 新增的pkg-config依赖有助于更好地管理构建时的依赖关系

改进建议:

  1. 日期问题:
  • changelog中的日期"Fri, 17 Oct 2025 08:35:29 +0800"是未来的日期,应该修改为当前实际日期
  1. 版本控制:
  • 建议在更新版本号时,同时更新所有相关文件中的版本引用,确保一致性
  1. 依赖管理:
  • pkg-config的添加是合理的,但建议在构建测试中验证所有依赖是否正确解析
  • 考虑添加最低版本的pkg-config要求,例如"pkg-config (>= 0.29)"
  1. 文档完整性:
  • 建议在changelog中添加更详细的变更说明,说明为什么需要调整pkg-config依赖
  1. 构建验证:
  • 建议在提交前进行完整的构建测试,确保新增依赖不会导致构建失败

总体而言,这次变更的内容是合理的,主要是版本更新和依赖调整,符合Debian包管理的基本规范。主要需要注意日期问题和确保依赖关系的正确性。

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, lzwind

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 72aebbe into linuxdeepin:master Oct 17, 2025
14 of 15 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