[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Christoph Feck]

https://bugs.kde.org/show_bug.cgi?id=366793

Christoph Feck  changed:

   What|Removed |Added

   See Also||https://bugs.kde.org/show_b
   ||ug.cgi?id=387375

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[John Haxby]

https://bugs.kde.org/show_bug.cgi?id=366793

John Haxby  changed:

   What|Removed |Added

 CC||john.ha...@oracle.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer]

https://bugs.kde.org/show_bug.cgi?id=366793

Wolfgang Bauer  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit||https://cgit.kde.org/konsol
   ||e.git/commit/?id=a779c8314a
   ||7d27bb4691c220f793540e6f84f
   ||6c9
 Status|CONFIRMED   |RESOLVED
   Version Fixed In||17.04.0

--- Comment #27 from Wolfgang Bauer  ---
The fix has been committed yesterday:
https://cgit.kde.org/konsole.git/commit/?id=a779c8314a7d27bb4691c220f793540e6f84f6c9

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Adam]

https://bugs.kde.org/show_bug.cgi?id=366793

Adam  changed:

   What|Removed |Added

 CC||k...@xibo.at

--- Comment #26 from Adam  ---
Also prevents usage of konsole as a terminal in QtCreator when parameters are
present.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Alex Bikadorov]

https://bugs.kde.org/show_bug.cgi?id=366793

Alex Bikadorov  changed:

   What|Removed |Added

 CC||krugovy...@mail.ru

--- Comment #25 from Alex Bikadorov  ---
*** Bug 374625 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Jim Jones]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #24 from Jim Jones  ---
> Btw, regarding *this* bug, somebody proposed a patch, which I can confirm is
> fixing the problems with the '-e' parameter:
> https://git.reviewboard.kde.org/r/129677/

thanks, it works for me too

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #23 from Wolfgang Bauer  ---
(In reply to OndraK from comment #22)
> Steps to reproduce:
> 1. define global shortcut, e.g., Win+v in systemsettings > global shortcuts,
> that would run command 'konsole -e vim' (w/o single quotes)
> 2. use Win+v to launch vim in konsole
> 3. start another instance of konsole (not tab), either A-F2, launch menu,
> quick launch widget
> 
> Actual result:
> konsole is started with vim launched; Works with mc, ncmpcpp. When I go to
> 'Edit current profile', 'Command' field in 'General' tab does state 'vim'
> instead of full path to shell executable

This rather sounds like bug#371863 (and its duplicates), which should be fixed
in 16.12.0.

Btw, regarding *this* bug, somebody proposed a patch, which I can confirm is
fixing the problems with the '-e' parameter:
https://git.reviewboard.kde.org/r/129677/

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[OndraK]

https://bugs.kde.org/show_bug.cgi?id=366793

OndraK  changed:

   What|Removed |Added

 CC||ondrej.kaji...@gmail.com

--- Comment #22 from OndraK  ---
Hello,

don't know if my problem is related to this bug, but might be:

Reproducible: always

Steps to reproduce:
1. define global shortcut, e.g., Win+v in systemsettings > global shortcuts,
that would run command 'konsole -e vim' (w/o single quotes)
2. use Win+v to launch vim in konsole
3. start another instance of konsole (not tab), either A-F2, launch menu, quick
launch widget

Actual result:
konsole is started with vim launched; Works with mc, ncmpcpp. When I go to
'Edit current profile', 'Command' field in 'General' tab does state 'vim'
instead of full path to shell executable

Expected result:
Having konsole launched with shell only

Running konsole 16.08.3-r1

However, when the command is 'konsole --name Vim -e vim', next konsole I start
works fone. So, this problem can be at least work-arounded :)

Thank you all who have ever worked on KDE :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #21 from Wolfgang Bauer  ---
(In reply to Jonathan Marten from comment #20)
> Some of
> the required functionality can be restored by setting the parser to not
> parse options after the first non-option argument:

Yes, but this requires Qt 5.6 at least.
Probably not a problem for KDE Applications 17.04, but it will prevent
distributions that still use Qt 5.5 to backport the fix (I don't know if any of
them ship konsole 16.08 though).

> The workaround is to ensure that the first argument is not an option:

Another workaround would be to make the '-e' option not take any arguments at
all, then the actual command would be the first positional argument.
(disadvantage: QCommandLineParser will not complain if there is no command at
all after '-e', and the command itself may not start with a '-' but that can be
ignored I suppose)

Another way I see to fix the problem is to parse the command line manually
before handing it over to QCommandLineParser, and strip off '-e' and all that
follows (and handle it manually).

Actually I experimented myself with both a bit already, but had no time yet to
get it into a fully working shape...

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Jonathan Marten]

https://bugs.kde.org/show_bug.cgi?id=366793

Jonathan Marten  changed:

   What|Removed |Added

 CC||j...@keelhaul.me.uk

--- Comment #20 from Jonathan Marten  ---
The cause appears to be a shortcoming in QCommandLineParser in that it has no
option syntax for "capture all the remaining arguments after this one", as was
the case for KCmdLineArgs and an option starting with '!'.  Some of the
required functionality can be restored by setting the parser to not parse
options after the first non-option argument:

--- a/src/main.cpp
+++ b/src/main.cpp
@@ -109,6 +109,7 @@ extern "C" int Q_DECL_EXPORT kdemain(int argc, char*
argv[])

 QSharedPointer parser(new QCommandLineParser);
 parser->setApplicationDescription(about.shortDescription());
+   
parser->setOptionsAfterPositionalArgumentsMode(QCommandLineParser::ParseAsPositionalArguments);
 parser->addHelpOption();
 parser->addVersionOption();
 about.setupCommandLine(parser.data());

but which still fails if the first option after the command name is an option:

konsole --hold -e ls -l main.cpp   => error "Unknown option 'l'."

The workaround is to ensure that the first argument is not an option:

konsole --hold -e ls main.cpp -l   => works

which only works if the command accepts options after non-option arguments;
alternatively

konsole --hold -e ls -- -l main.cpp=> works

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Christian González]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #19 from Christian González  ---
Will this bug be fixed before 16.12 (currently in beta) releasing? Or we will
have to wait to 17.04 (or beyond)?

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=366793

andydecle...@gmail.com changed:

   What|Removed |Added

 CC||andydecle...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Petr Vorálek]

https://bugs.kde.org/show_bug.cgi?id=366793

Petr Vorálek  changed:

   What|Removed |Added

 CC||nazir...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Tobias Hunger via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Tobias Hunger  changed:

   What|Removed |Added

 CC||tobias.hun...@gmail.com

--- Comment #18 from Tobias Hunger  ---
Indeed the command line parsing of konsole is broken. It parses any option
passed after -e as its own. E.g. "konsole -e echo --authors" makes konsole dump
the list of its authors.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Piotr Dobrogost via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Piotr Dobrogost  changed:

   What|Removed |Added

 CC||p...@bugs.kde.dobrogost.net

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Aki Koskinen via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Aki Koskinen  changed:

   What|Removed |Added

 CC||k...@akikoskinen.info

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Vitor M . Pereira via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Vitor M. Pereira  changed:

   What|Removed |Added

 CC||vmper...@fc.up.pt

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Antonio Rojas via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Antonio Rojas  changed:

   What|Removed |Added

 CC||jesa...@gmail.com

--- Comment #17 from Antonio Rojas  ---
*** Bug 369314 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Wolfgang Bauer  changed:

   What|Removed |Added

 CC||brez...@gmail.com

--- Comment #16 from Wolfgang Bauer  ---
*** Bug 369297 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Antonio Rojas via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Antonio Rojas  changed:

   What|Removed |Added

 CC||ila...@gmail.com

--- Comment #15 from Antonio Rojas  ---
*** Bug 368899 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Antonio Rojas via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Antonio Rojas  changed:

   What|Removed |Added

 CC||chris.chap...@gmail.com

--- Comment #14 from Antonio Rojas  ---
*** Bug 369040 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Kevin Kofler via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #13 from Kevin Kofler  ---
> This breaks at least the debuginfo-install.sh from kdelibs4.

>From kde(4)-runtime's DrKonqi, actually.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Kevin Kofler via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #12 from Kevin Kofler  ---
It's not the same regression (but likely also a consequence of the port): Here,
we DON'T want to have -c parsed by Konsole at all, it should be treated as an
argument to -e.

This breaks at least the debuginfo-install.sh from kdelibs4.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Robin Green via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Robin Green  changed:

   What|Removed |Added

 CC||gree...@greenrd.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Rex Dieter via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #11 from Rex Dieter  ---
appears to be a consequence of porting away from kdelibs4support, which
provided support for a plethora of extra command line options, including:
-caption (see bug #368949)

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Rex Dieter via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Rex Dieter  changed:

   What|Removed |Added

   Keywords||regression
   Severity|normal  |major

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Rex Dieter via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Rex Dieter  changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 Ever confirmed|0   |1

--- Comment #10 from Rex Dieter  ---
Marking confirmed.

Another side-effect is that application desktop shortcuts that include
Terminal=true
no longer work either (see also bug #368949)

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

svadk...@gmail.com changed:

   What|Removed |Added

 CC||svadk...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #9 from rauchwo...@gmx.net ---
commands like 

"konsole -e mpv --fs /mnt/foo.avi also don't work"

--fs is used by konsole and i get Unknown option 'fs'.
although konsole -e "mpv --fs /mnt/foo.avi" is working.

Another combination that doesnt work are spaces inside filenames until they
spaces are escaped.

konsole -e "mpv /mnt/data/foo\ 1.mp4" - works
konsole -e "mpv /mnt/data/foo 1.mp4" - doesn't work

please get this fixed and provide the functionality descibed in the help
output:

Options: ... -e  Command to execute. This option will catch all following
arguments, so use it as the last option.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[stephane via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

stephane  changed:

   What|Removed |Added

 CC||steph...@scachemaille.ch

--- Comment #8 from stephane  ---
I don't know if it's related. 
but on manjaro (arch derivative) since konsole 16.08 octopi (graphical package
installer) don't work when it call terminal to install AUR package. seems
konsole don't start correctly.
and also someone reported that his shortcut was not working anymore.

downgrading konsole to 16.04 solve the problem.

https://forum.manjaro.org/t/unstable-update-2016-08-30-networkmanager-haskell-plasma-5-7-4/8371/37

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Rex Dieter via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Rex Dieter  changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Simon Andric via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Simon Andric  changed:

   What|Removed |Added

 CC||simonandr...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Wolfgang Bauer  changed:

   What|Removed |Added

 CC||rauchwo...@gmx.net

--- Comment #7 from Wolfgang Bauer  ---
*** Bug 367588 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Michael Palimaka via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Michael Palimaka  changed:

   What|Removed |Added

 CC||kensing...@gentoo.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #6 from Wolfgang Bauer  ---
Yes, for SSH ktelnetservice5 runs "konsole --noclose -e ssh -l username host -p
port" (if a username or port is not specified, the corresponding option is
omitted),
So this is indeed likely the same issue.

ktelnetservice5 does take the default terminal application from systemsettings
though, so you could also change that to xterm as work-around.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Christian González via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #5 from Christian González  ---
It seems that this bug is also affecting URLs with the "ssh://user@server"
schema.

For example: I use KeePassX2 to store all users, passwords and URLs (http//,
ssh//, etc.) for all my servers. When I click on a URL, it launches "xdg-open
ssh://user@server" and in turn xdg-open launches "ktelnetservice5
ssh://user@server". Before 16.08.0, Konsole was launched with a SSH session
established. Now, ktelnetservice5 just end without results. (In one of my test
it shows the message "Unknown option 'l'" but I can't reproduce it anymore. I
guess the 'l' option means the 'l' parameter from ssh client so maybe
ktelnetservice5 is using the "ssh -l user server" sintax and it raises this
bug.)

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Christian González via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Christian González  changed:

   What|Removed |Added

 CC||chgonzal...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #4 from Wolfgang Bauer  ---
Btw, this works as expected:
konsole -e echo test
(or konsole --hold -e echo test)
or even:
konsole -e echo This is a test

Both print the given text in the new konsole window.

So the problem only seem to occur if there is another option (with '-')
following the -e.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #3 from Wolfgang Bauer  ---
(In reply to Raymond Wooninck from comment #2)
> It seems as parsing the command line is the issue and that everything is
> seen as an option to konsole.

Yes, obviously.
But -e should "capture" all following options, and it does in previous konsole
versions (which used kdelibs4support, i.e. KCmdLineArgs, for parsing the
command line).

>  Using konsole -e "sh -C "echo test"", then it
> works fine.

No, it doesn't.
This displays the following error message in the new konsole window:
/usr/bin/test: /usr/bin/test: cannot execute binary file

konsole -e "sh -c \"echo test\"" would work though.

Btw:
(In reply to Wolfgang Bauer from comment #0)
> (of course it would close immediately again, adding the "--keep" parameter
> should fix that)
I wanted to write "--hold" of course, not "keep".

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Raymond Wooninck via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

Raymond Wooninck  changed:

   What|Removed |Added

 CC||tittiatc...@gmail.com

--- Comment #2 from Raymond Wooninck  ---
It seems as parsing the command line is the issue and that everything is seen
as an option to konsole.  Using konsole -e "sh -C "echo test"", then it works
fine.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

edisonalvari...@bol.com.br changed:

   What|Removed |Added

 CC||edisonalvari...@bol.com.br

-- 
You are receiving this mail because:
You are watching all bug changes.

[konsole] [Bug 366793] Regression in Konsole's handling of command line parameters

[Wolfgang Bauer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=366793

--- Comment #1 from Wolfgang Bauer  ---
I'd like to add that konsole 16.08.0's command line help still says this:
$ konsole --help
Usage: konsole [options] [args]
Terminal emulator

Options:
...
  -eCommand to execute. This option will catch all
 following arguments, so use it as the last option.
...

So according to the documentation, this is supposed to work... ;-)

-- 
You are receiving this mail because:
You are watching all bug changes.