chore(debian): exclude platform wallpapers from dh_install#57
Conversation
1. Added override_dh_install to skip platform-specific wallpaper directories Log: Exclude deepin/platform and deepin-livewallpapers/platform from install chore(debian): 在 dh_install 中排除 platform 壁纸目录 1. 新增 override_dh_install 以跳过平台特定的壁纸目录 Log: 安装时排除 deepin/platform 和 deepin-livewallpapers/platform 目录
deepin pr auto review你好!我是CodeGeeX。我已仔细审查了你提供的 这个修改的主要目的是在 Debian 打包的 以下是我对这段代码的详细审查意见: 1. 语法逻辑
2. 代码质量
3. 代码性能
4. 代码安全
💡 改进建议1. 提升可读性与可维护性(换行与续行符) override_dh_install:
dh_install \
--exclude=deepin/platform \
--exclude=deepin-livewallpapers/platform2. 考虑更精确的路径匹配(如果需要) override_dh_install:
dh_install
# 显式删除特定目录,更加精确且意图明确
rm -rf debian/your-package-name/usr/lib/deepin/platform
rm -rf debian/your-package-name/usr/lib/deepin-livewallpapers/platform注意:使用 3. 添加注释说明意图 # 排除 deepin 相关平台插件,因为 [填写排除原因,如:许可证问题/依赖缺失等]
override_dh_install:
dh_install --exclude=deepin/platform --exclude=deepin-livewallpapers/platform总结:当前的修改在语法和逻辑上是正确且可用的,主要建议在于增加注释说明排除原因,并在未来排除项增多时注意换行排版。 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, mhduiy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Log: Exclude deepin/platform and deepin-livewallpapers/platform from install
chore(debian): 在 dh_install 中排除 platform 壁纸目录
Log: 安装时排除 deepin/platform 和 deepin-livewallpapers/platform 目录