Skip to content

Conversation

@dengzhongyuan365-dev
Copy link
Contributor

merge to master

deepin-admin-bot and others added 25 commits May 10, 2023 02:19
sync by github workflow

Log: none
Influence: none
sync by github workflow

Log: none
Influence: none
Log: opencv 引入时使用 3rdparty/opencv-4.5.4/build/install/lib/libopencv_world.a,写死了 lib 路径,但是 CMAKE_INSTALL_LIBDIR 在一些 64 位系统中是 lib64
  安装路径Prefix规范化

Log: 安装路径Prefix规范化
后端切换至dtkocr

Log: 后端切换至dtkocr
更新翻译文件

Log: 更新翻译文件
add OBS workflows

log:
修改changelog

Log: 修改changelog
添加翻译配置文件

Log: 修复部分已知问题
sync by github workflow

Log: none
Influence: none
sync by github workflow

Log: none
Influence: none
Update changelog

Log: Update changelog
Fix wrong combobox window palette

Log: Fix wrong combobox window palette
Bug: https://pms.uniontech.com/bug-view-241599.html
Adapt Treeland, disable wayland settings in
Treeland enviroment.

Log: Adapt Treeland.
update version

Log: update version
Fixed the Compact Mode issue

Bug: https://pms.uniontech.com/bug-view-202377.html
Log: Fixed the Compact Mode issue
rmove kwayland-shell

Log: remove kwayland-shell
Optimized vulkan dependencies

Log: Optimized vulkan dependencies
feat: 添加 DConfig 相关源代码。
On some device models, using GPU acceleration during the OCR process leads to crashes during destruction. Now, the os-config solution is employed to customize these machines.

fix: 某些机型上,使用GPU加速导致崩溃

在某些机型上,OCR过程中使用GPU加速,析构时会导致崩溃。现在使用os-config方案对这些机器进行定制。
When performing the destruction operation, it is necessary to wait for the child thread to exit to avoid errors in resource release.

fix: 关闭应用引起进程崩溃

析构操作时,需要等待子线程退出,避免资源释放出错。

Bug: https://pms.uniontech.com/bug-view-334545.html
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

TAG Bot

TAG: 6.5.14
EXISTED: no
DISTRIBUTION: unstable

@dengzhongyuan365-dev dengzhongyuan365-dev force-pushed the master branch 2 times, most recently from 6813379 to 29b7d0a Compare December 2, 2025 06:26
Merge release/eagle into master, bringing in:
- Treeland support and disabling wayland settings in Treeland
- GPU acceleration crash fixes
- DConfig backend and config integration
- UI and translation fixes from release/eagle
- Added libdtk6ocr-dev as alternative to libdtkocr-dev in Debian build dependencies
- Added utils directory to CMake include paths and source directories
- Updated translation file line number references across all language files to reflect code changes

Log: Update build dependencies and translation references
Sync local master with upstream/master:
- Pick up upstream 6.5.13 changes
- Align Debian Build-Depends and CI configuration
- Ensure later patches are based on the latest upstream
- Added SPDX-FileCopyrightText for UnionTech Software Technology Co., Ltd.
- Added SPDX-License-Identifier CC0-1.0 to CMake configuration file

Log: Add license header to CMake configuration
- Merged release/eagle branch into master
- Updated Debian changelog with new version entry

Log: update version to 6.5.14
@deepin-ci-robot
Copy link

deepin pr auto review

我来对这个diff进行代码审查。主要关注以下几个方面:

  1. 语法逻辑
  2. 代码质量
  3. 代码性能
  4. 代码安全

让我们先分析一下主要变更:

  1. 添加了GPU加速配置功能
  2. 优化了构建配置
  3. 添加了新的翻译文件
  4. 修改了UI布局

以下是我的审查意见:

  1. 语法逻辑
  • 代码语法正确,没有发现明显的语法错误
  • GPU加速的逻辑判断合理,通过配置文件控制
  1. 代码质量
  • 配置文件管理采用了单例模式,设计合理
  • UI布局代码组织良好,但有一些重复代码可以优化
  1. 代码性能
  • GPU加速的判断逻辑高效
  • QProcess的使用合理,但建议添加超时处理
  1. 代码安全
  • 配置文件权限设置为"readwrite",建议根据实际需求调整为更严格的权限
  • 进程执行没有做输入验证,存在安全风险

具体建议:

  1. GPU加速相关代码:
// 建议添加超时处理
kxProc.setProcessTimeout(5000); // 5秒超时
if (!kxProc.waitForFinished(5000)) {
    kxProc.kill();
    qWarning() << "CPU info detection timeout";
}
  1. 配置文件权限:
"permissions": "read"  // 如果只需要读取,建议改为read
  1. UI布局优化:
// 建议抽取公共代码
auto createButton(const QString& icon, const QString& tooltip) {
    auto btn = new QPushButton;
    btn->setIcon(DDciIcon(icon));
    btn->setMaximumSize(36, 36);
    btn->setIconSize(QSize(14, 14));
    btn->setToolTip(tooltip);
    return btn;
}
  1. 翻译文件:
  • 新增的翻译条目完整
  • 建议使用统一的翻译工具确保格式一致性
  1. 构建系统:
  • CMake配置合理,支持条件编译
  • 建议添加版本检查:
if(CMAKE_VERSION VERSION_LESS "3.16")
    message(FATAL_ERROR "CMake 3.16 or higher is required")
endif()

总体来说,这次变更质量良好,主要是一些细节优化建议。

@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

@dengzhongyuan365-dev
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 2, 2025

This pr force merged! (status: unstable)

@dengzhongyuan365-dev
Copy link
Contributor Author

merge

@dengzhongyuan365-dev
Copy link
Contributor Author

/merge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 2, 2025

This pr cannot be merged! (status: unstable)

@dengzhongyuan365-dev
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 2, 2025

This pr force merged! (status: unstable)

@dengzhongyuan365-dev
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Dec 2, 2025

This pr force merged! (status: unstable)

@lzwind lzwind merged commit 69e041a into linuxdeepin:master Dec 2, 2025
18 of 19 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.