Bug 495564

Summary:	merge excess config files
Product:	[Plasma] KRdp	Reporter:	Harald Sitter <sitter>
Component:	general	Assignee:	Unassigned bugs mailing-list <unassigned-bugs>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	ahiemstra, kde, nate
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/krdp/-/commit/332834b1ecbd2ce70744330d635fe0c9f371cd20	Version Fixed In:	6.3.0
Sentry Crash Report:

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