Bug 495564 - merge excess config files
Summary: merge excess config files
Status: RESOLVED FIXED
Alias: None
Product: KRdp
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-10-30 05:22 UTC by Harald Sitter
Modified: 2024-11-01 13:12 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.3.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Sitter 2024-10-30 05:22:16 UTC
SUMMARY
krdpserver appears to have two config files. It might make sense to merge them ;)

STEPS TO REPRODUCE
1.  configure in kcm (creates krdpserverrc)
2.  do connect flow (creates krdp-serverrc with restoreid)

OBSERVED RESULT
❯ ls ~/.config/krdp*
/home/me/.config/krdp-serverrc  /home/me/.config/krdpserverrc


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Operating System: KDE Linux e4a55271
KDE Plasma Version: 6.2.80
KDE Frameworks Version: 6.8.0
Qt Version: 6.8.0
Kernel Version: 6.11.4-arch2-1
Graphics Platform: Wayland
Processors: 32 × AMD Ryzen 9 5950X 16-Core Processor
Memory: 31.2 GiB of RAM
Graphics Processor: AMD Radeon RX 7800 XT

ADDITIONAL INFORMATION
Comment 1 Bug Janitor Service 2024-11-01 10:10:24 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/krdp/-/merge_requests/70
Comment 2 David Edmundson 2024-11-01 11:29:52 UTC
Git commit 332834b1ecbd2ce70744330d635fe0c9f371cd20 by David Edmundson.
Committed on 01/11/2024 at 10:12.
Pushed by davidedmundson into branch 'master'.

Use single config file for krdpserver

The kcfg file for krdp server is "krdpserver" the app name is
krdp-server.

This led to all settings being in krdpserver except the restore token
which blindly used KSharedConfig.

A compatibility path for reading is left in place that can be dropped
later.

M  +11   -3    src/PortalSession.cpp

https://invent.kde.org/plasma/krdp/-/commit/332834b1ecbd2ce70744330d635fe0c9f371cd20