Bug 458411

Summary: Integrated terminal in Dolphin should use \r as line terminator instead of \n
Product: [Applications] dolphin Reporter: Yesterday17 <mmf>
Component: panels: terminalAssignee: Dolphin Bug Assignee <dolphin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: kfm-devel, nate
Priority: NOR    
Version: 22.04.3   
Target Milestone: ---   
Platform: Manjaro   
OS: Linux   
Latest Commit: https://invent.kde.org/system/dolphin/commit/dfe05c587beab876c0bf0718b84c7ed14bafde95 Version Fixed In:
Sentry Crash Report:

Description Yesterday17 2022-08-28 08:15:10 UTC 
SUMMARY

The integrated terminal in Dolphin executes `cd` and `clear` commands with `\n` as line terminator. The `\n` here should be replaced with `\r` as the behavior of Konsole.

Corresponding issue:
https://github.com/nushell/nushell/issues/6427

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux
KDE Plasma Version: 5.24.6
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Kernel Version: 5.15.60-1-MANJARO (64-bit)
Graphics Platform: X11
Processors: 32 × AMD Ryzen 9 5950X 16-Core Processor
Memory: 62.7 GiB of RAM
Graphics Processor: NVIDIA GeForce RTX 3070 Ti/PCIe/SSE2

ADDITIONAL INFORMATION
Comment 1 Bug Janitor Service 2022-08-28 09:13:42 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/system/dolphin/-/merge_requests/441
Comment 2 ratijas 2022-11-23 20:05:01 UTC 
Git commit dfe05c587beab876c0bf0718b84c7ed14bafde95 by ivan tkachenko, on behalf of Haozhe Jiang.
Committed on 23/11/2022 at 17:46.
Pushed by felixernst into branch 'master'.

Use \r to terminate command in integrated terminal

\r should be used instead of \n

M  +2    -2    src/panels/terminal/terminalpanel.cpp

https://invent.kde.org/system/dolphin/commit/dfe05c587beab876c0bf0718b84c7ed14bafde95