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

KDE wallet: ship KDE distro with default disabled KDE Wallet #7302

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

igorpecovnik
Copy link
Member

@igorpecovnik igorpecovnik commented Sep 26, 2024

Description

It is better to store WiFi passwords (and similar) encrypted. Which is one of the advantages of Kwallet. But since this is not the case with other desktops, lets set it to disabled by default. People who use it,

kwallet -> enable kwallet subsystem.

How Has This Been Tested?

  • Produced KDE image comes with disabled subsystem

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@igorpecovnik igorpecovnik requested a review from a team as a code owner September 26, 2024 12:23
@github-actions github-actions bot added size/small PR with less then 50 lines BSP Board Support Packages labels Sep 26, 2024
@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Needs review Seeking for review labels Sep 26, 2024
@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Sep 29, 2024
@igorpecovnik igorpecovnik merged commit 2626dcc into main Sep 29, 2024
5 checks passed
@igorpecovnik igorpecovnik deleted the kde-wallet branch September 29, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release BSP Board Support Packages Needs review Seeking for review Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants