fix: add executable bit for deepin-greater

[Cryolitia]

[deepin-ci-robot]

deepin pr auto review

代码审查意见如下：

1. CMakeLists.txt 文件修改：

   • 在 install 命令中，将 FILES 改为 PROGRAMS 是正确的，因为 deepin-greeter 文件是一个可执行程序。这个修改提高了代码的可读性和准确性。

2. 文件路径和目标路径：

   • 确保所有安装路径（如 ${CMAKE_INSTALL_BINDIR}、${CMAKE_INSTALL_DATADIR}、${CMAKE_INSTALL_SYSCONFDIR}）在项目配置中已经正确设置，并且这些路径在目标系统上是可访问的。

3. 文件 globbing：

   • 使用 file(GLOB ...) 来获取脚本文件时，建议添加一个条件来限制搜索范围，避免不必要的文件被包含。例如，可以指定一个特定的目录或文件模式。

4. 代码注释和文档：

   • 虽然这个提交没有添加新的注释，但建议在 CMakeLists.txt 文件中添加一些注释来解释为什么需要这些安装命令，以及这些文件的作用。这对于维护代码的人来说非常有帮助。

5. 一致性：

   • 确保整个 CMakeLists.txt 文件中的安装命令使用一致的方式，例如，如果某些文件是可执行程序，那么所有相关的文件都应该使用 PROGRAMS 关键字。

6. 错误处理：

   • 虽然这个提交没有涉及到错误处理，但在更复杂的 CMake 项目中，建议添加一些错误处理逻辑，例如检查文件是否存在，或者安装目标是否成功。

7. 性能考虑：

   • 如果 file(GLOB ...) 命令会返回大量文件，建议评估其性能影响，并考虑使用其他方法来指定要安装的文件。

总体来说，这个提交的修改是合理的，但建议在代码中添加一些额外的注释和错误处理逻辑，以提高代码的可维护性和健壮性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, Cryolitia, yixinshark

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

[yixinshark]

@18202781743 周末起来挺早的呀 😄