Skip to content

fix: fix translation generation and qm install path in CMakeLists - #222

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:release/1071from
GongHeng2017:202609141317-re-1071-fix
Sep 14, 2026
Merged

deepin-bot[bot] merged 1 commit into
linuxdeepin:release/1071from
GongHeng2017:202609141317-re-1071-fix

Conversation

@GongHeng2017

Copy link
Copy Markdown
Contributor

Replace inline lupdate/qt5_create_translation with translate_generation.sh script for qm generation. Fix WORKING_DIRECTORY and qm glob path to use CMAKE_CURRENT_SOURCE_DIR instead of CMAKE_SOURCE_DIR. Remove extra spaces in "Available:" translation text.

用 translate_generation.sh 脚本替代内联 lupdate/qt5_create_translation 逻辑 修复 WORKING_DIRECTORY 和 qm 文件路径使用 CMAKE_CURRENT_SOURCE_DIR 修正 "Available:" 翻译文本中多余空格

Log: 修复翻译文件生成路径错误及翻译文本格式问题
PMS: BUG-204631
Influence: 修复构建时找不到翻译脚本的报错,确保qm文件正确生成和安装

Replace inline lupdate/qt5_create_translation with translate_generation.sh
script for qm generation. Fix WORKING_DIRECTORY and qm glob path to use
CMAKE_CURRENT_SOURCE_DIR instead of CMAKE_SOURCE_DIR. Remove extra spaces
in "Available:" translation text.

用 translate_generation.sh 脚本替代内联 lupdate/qt5_create_translation 逻辑
修复 WORKING_DIRECTORY 和 qm 文件路径使用 CMAKE_CURRENT_SOURCE_DIR
修正 "Available:" 翻译文本中多余空格

Log: 修复翻译文件生成路径错误及翻译文本格式问题
PMS: BUG-204631
Influence: 修复构建时找不到翻译脚本的报错,确保qm文件正确生成和安装

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @GongHeng2017, your pull request is larger than the review limit of 150,000 diff characters

@GongHeng2017

Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

This pr force merged! (status: unstable)

@deepin-bot
deepin-bot Bot merged commit f1e2982 into linuxdeepin:release/1071 Sep 14, 2026
15 of 16 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.

2 participants