Skip to content

chore: add polkit policy to debian install#586

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
52cyb:master
Jul 27, 2026
Merged

chore: add polkit policy to debian install#586
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
52cyb:master

Conversation

@52cyb

@52cyb 52cyb commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Add the polkit policy action file to the libdde-network-core package to ensure correct installation and enable proper privilege escalation for network management operations.

Log: Added polkit policy file for network management

Influence:

  1. Verify that the polkit policy file is installed in the correct location: /usr/share/polkit-1/actions/com.deepin.dde.network.policy
  2. Test that network operations requiring privileges now work correctly (e.g., changing network settings, connecting to networks)
  3. Ensure no existing functionality is broken by the addition of the policy file

chore: 添加 polkit 策略到 debian 安装

将 polkit 策略配置文件添加入 libdde-network-core 包中,确保其能被正确安 装,从而为网络管理操作提供正确的权限提升支持。

Log: 新增网络管理 polkit 策略文件
PMS: BUG-355287
Influence:

  1. 确认 polkit 策略文件被安装在正确位置:/usr/share/polkit-1/actions/ com.deepin.dde.network.policy
  2. 测试需要特权的网络操作是否能正常工作(例如:更改网络设置、连接网 络等)
  3. 确保添加策略文件不会破坏现有功能

Summary by Sourcery

Build:

  • Include the com.deepin.dde.network.policy polkit action file in the libdde-network-core Debian package installation paths.

Add the polkit policy action file to the libdde-network-core package to
ensure correct installation and enable proper privilege escalation for
network management operations.

Log: Added polkit policy file for network management

Influence:
1. Verify that the polkit policy file is installed in the correct
location: /usr/share/polkit-1/actions/com.deepin.dde.network.policy
2. Test that network operations requiring privileges now work correctly
(e.g., changing network settings, connecting to networks)
3. Ensure no existing functionality is broken by the addition of the
policy file

chore: 添加 polkit 策略到 debian 安装

将 polkit 策略配置文件添加入 libdde-network-core 包中,确保其能被正确安
装,从而为网络管理操作提供正确的权限提升支持。

Log: 新增网络管理 polkit 策略文件
PMS: BUG-355287
Influence:
1. 确认 polkit 策略文件被安装在正确位置:/usr/share/polkit-1/actions/
com.deepin.dde.network.policy
2. 测试需要特权的网络操作是否能正常工作(例如:更改网络设置、连接网
络等)
3. 确保添加策略文件不会破坏现有功能
@deepin-ci-robot

Copy link
Copy Markdown

Hi @52cyb. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 52cyb, caixr23

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

Details 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

@52cyb

52cyb commented Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

This pr force merged! (status: unstable)

@deepin-bot
deepin-bot Bot merged commit 2779aee into linuxdeepin:master Jul 27, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants