Bug 156826 - quotes in profile command are not kept
Summary: quotes in profile command are not kept
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Robert Knight
URL:
Keywords:
: 163839 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-01-28 02:26 UTC by Laurence
Modified: 2008-09-28 22:13 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Laurence 2008-01-28 02:26:56 UTC
Version:           2.1 (using Devel)
Installed from:    Compiled sources
Compiler:          gcc 4.1.2 
OS:                Linux

In the "Edit profile..."-dialog, when I try to set a command with quotes or double quotes in them, the quotes are not saved (but it still works in the current konsole). So, when I set set command: su -c "screen -R" ,I can start that session in the current konsole, but when I go back to the profile-editing-dialog, the quotes are gone, and the session won't work after restarting konsole.
Comment 1 Steven Schwartfeger 2008-01-28 02:51:36 UTC
Finally found the report :) I don't know how the process here goes, but I can confirm this, as I use su -c "screen -R -S Admin" often.

The version is whatever comes with kde-4.0.0, but from comparing my source to HEAD it looks the same, quotes are stripped out for some reason (in ShellCommand.cpp).
Comment 2 Robert Knight 2008-01-30 01:43:55 UTC
> quotes are stripped out for some reason

The arguments are stored as separate strings internally, so the quotes are not necessary.  Plus arguments need to be unquoted when passing them as the command-line to the new process.
Comment 3 Armin Berres 2008-02-07 20:49:27 UTC
Yay, just wanted to report exactly this bug.
I have a profile to open a screen session on a remote host. The command I execute is something like this: "autossh -t foo 'screen -raAd'". Without the ' the call is quite useless.
I edited the .profile file by hand and everything works fine now.
Comment 4 Robert Knight 2008-06-12 04:49:02 UTC
*** Bug 163839 has been marked as a duplicate of this bug. ***
Comment 5 Robert Knight 2008-07-25 17:05:00 UTC
SVN commit 837723 by knight:

Quote arguments containing spaces when expanding the list of arguments
in a ShellCommand.

BUG: 156826
BACKPORT: 4.1


 M  +12 -1     ShellCommand.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=837723