fix: 修复旧版控制中心无法加入内测的问题

linuxdeepin / lastore-daemon

GNU General Public License v3.0

12 stars 30 forks source link

fix: 修复旧版控制中心无法加入内测的问题 #129

Closed ArchieMeng closed 1 month ago

ArchieMeng commented 1 month ago

将其加入允许安装的列表

Bug: https://pms.uniontech.com/bug-view-273975.html Log: 修复旧版控制中心无法加入内测的问题

deepin-ci-robot commented 1 month ago

deepin pr auto review

关键摘要:

在新增的常量 oldControlCenterPath 中，路径格式可能不正确，应该检查是否需要添加文件扩展名。
allowRemovePackageExecPaths 变量中的字符串应该使用 strings.Builder 来构建，以提高性能。
检查 strv.Strv 类型是否已经包含了 oldControlCenterPath，避免重复添加。

是否建议立即修改:

ArchieMeng commented 1 month ago

/recheck

ArchieMeng commented 1 month ago

/retest

deepin-ci-robot commented 1 month ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ArchieMeng, zsien

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/lastore-daemon/blob/master/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