sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#524

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[deepin-ci-robot]

deepin pr auto review

我来对这个diff进行审查：

1. 语法逻辑：

• 修改是合法的CMake语法，从代码结构上看没有语法错误
• install命令的参数格式正确

2. 代码质量：

• 这个修改降低了代码的可维护性和清晰度
• 原来的路径"${INCLUDE_INSTALL_DIR}/DCore/global/"更具体，表明了文件的明确位置
• 新的路径"${INCLUDE_INSTALL_DIR}"过于宽泛，可能会造成文件组织混乱

3. 代码性能：

• 这个修改对性能没有显著影响，因为它只影响安装时的文件路径

4. 代码安全：

• 将文件直接安装到include根目录可能会增加文件冲突的风险
• 更具体的目录结构有助于避免命名冲突和意外覆盖

改进建议：

1. 建议保持原有的路径结构，即：

install(FILES ${CONFIGNAME} DESTINATION "${INCLUDE_INSTALL_DIR}/DCore/global/")

2. 如果确实需要修改安装路径，建议：

• 提供详细的修改理由
• 确保新路径与其他组件保持一致性
• 考虑添加注释说明路径变更的原因

3. 如果是为了简化路径，可以考虑：

set(DTK_INCLUDE_INSTALL_DIR "${INCLUDE_INSTALL_DIR}/DCore")
install(FILES ${CONFIGNAME} DESTINATION "${DTK_INCLUDE_INSTALL_DIR}/global")

这样既保持了清晰的目录结构，又便于统一管理DTK相关的头文件路径。