Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: dde-control-center can be uninstalled #281

Merged
merged 1 commit into from
May 14, 2024

Conversation

18202781743
Copy link
Contributor

dde-control-center's desktop has changed in commit
linuxdeepin/dde-control-center#1624

dde-control-center's desktop has changed in commit
linuxdeepin/dde-control-center#1624
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 代码中的 desktopintegration.cpp 文件路径修改是否正确,需要确认新的路径是否存在于项目中。
  • 检查是否有其他文件或代码依赖于旧的桌面集成 ID,以确保修改不会导致其他部分出错。

是否建议立即修改:

  • 是,需要确保修改后的路径和 ID 正确无误,并且不会影响项目的其他部分。同时,建议进行全面的测试以验证修改后的代码是否仍然按预期工作。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, BLumia

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

Needs approval from an approver in each of these files:

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

@BLumia
Copy link
Member

BLumia commented May 14, 2024

备忘:RC发出去后集成appstream新版本,确保 launchpad 启用 appstream 支持。

@18202781743 18202781743 merged commit 60fbef9 into linuxdeepin:master May 14, 2024
10 checks passed
BLumia added a commit to BLumia/dde-launchpad that referenced this pull request May 17, 2024
之前由于仓库尚不存在 1.x 版本的 AppStream 软件包,所以尽管 debian
依赖写了对 appstreamqt 的依赖,但实际对应的支持是不会被启用的。现
AppStream 1.0.3 已进入 deepin 仓库,故开始强制要求 AppStream 依赖
存在。

与 linuxdeepin#281 有一定相关性(原为 AppStream 配置
不存在时提供的后备方案)。

Log:
deepin-bot bot pushed a commit that referenced this pull request May 21, 2024
之前由于仓库尚不存在 1.x 版本的 AppStream 软件包,所以尽管 debian
依赖写了对 appstreamqt 的依赖,但实际对应的支持是不会被启用的。现
AppStream 1.0.3 已进入 deepin 仓库,故开始强制要求 AppStream 依赖
存在。

与 #281 有一定相关性(原为 AppStream 配置
不存在时提供的后备方案)。

Log:
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.

3 participants