Skip to content

chore: update build dependencies and CMake configuration#60

Merged
lzwind merged 2 commits intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master
Jun 19, 2025
Merged

chore: update build dependencies and CMake configuration#60
lzwind merged 2 commits intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master

Conversation

@dengzhongyuan365-dev
Copy link
Contributor

chore: update build dependencies and CMake configuration

- Added libncnn-dev and libopencv-mobile-dev to debian/control
- Updated CMakeLists.txt to include additional compiler flags for better compatibility and warnings management

- Added libncnn-dev and libopencv-mobile-dev to debian/control
- Updated CMakeLists.txt to include additional compiler flags for better compatibility and warnings management
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • debian/changelog文件中,版本号从6.5.0.1更新到6.5.1.1,但未提供更新的具体内容。
  • debian/control文件中新增了libncnn-devlibopencv-mobile-dev作为依赖项,需要确认这些库的版本兼容性和必要性。
  • linglong.yaml文件中的版本号从6.5.0.1更新到6.5.1.1,但未提供更新的具体内容。
  • src/CMakeLists.txt文件中新增了一些编译选项,需要确保这些选项不会影响现有功能,并且与目标平台兼容。

是否建议立即修改:

  • 是,需要提供版本更新的具体内容,以便其他开发者了解更新的目的和影响。
  • 是,需要确认新增的依赖项的版本兼容性和必要性,避免潜在的编译问题或运行时错误。
  • 是,需要提供版本更新的具体内容,以便其他开发者了解更新的目的和影响。
  • 是,需要确保新增的编译选项不会影响现有功能,并且与目标平台兼容,以避免潜在的编译问题或运行时错误。

@github-actions
Copy link

TAG Bot

TAG: 6.5.2
EXISTED: no
DISTRIBUTION: unstable

@lzwind lzwind merged commit c6397bd into linuxdeepin:master Jun 19, 2025
15 of 18 checks passed
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev, 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

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