Re: [PATCH 0/2] KRDC: Fix command line argument handling

2016-05-28 Thread Luigi Toscano
Arno ha scritto: > Hi Burkhard, > > On 5/28/2016 9:06 AM, Burkhard Lück wrote: >> please upload your patch to https://git.reviewboard.kde.org and select the >> repositoriy krdc > > Thanks for the hint. Done: > https://git.reviewboard.kde.org/r/128032/ > I would also add the kdeutils group to th

Re: [PATCH 0/2] KRDC: Fix command line argument handling

2016-05-28 Thread Arno
Hi Burkhard, On 5/28/2016 9:06 AM, Burkhard Lück wrote: > please upload your patch to https://git.reviewboard.kde.org and select the > repositoriy krdc Thanks for the hint. Done: https://git.reviewboard.kde.org/r/128032/ -- Regards, Arno

Re: [PATCH 0/2] KRDC: Fix command line argument handling

2016-05-28 Thread Burkhard Lück
Am Samstag, 28. Mai 2016, 01:35:18 CEST schrieb Arno: > Hi, > > It just occurred to me that I didn't include a reproducer/test case: > Just try: > ~~ > krdc rdp:// > ~~ > with and without the patches. > The patches are for git://anongit.kde.org/krdc, btw. Hi Anrno, please upload your patch to ht

Re: [PATCH 0/2] KRDC: Fix command line argument handling

2016-05-27 Thread Arno
Hi, It just occurred to me that I didn't include a reproducer/test case: Just try: ~~ krdc rdp:// ~~ with and without the patches. The patches are for git://anongit.kde.org/krdc, btw. -- Regards, Arno.

[PATCH 0/2] KRDC: Fix command line argument handling

2016-05-27 Thread Arno Moeller
From: Arno Two small fixes for KRDC. The first patch fixes parsing Url parsing from the command line. When passing rdp://, url.scheme() is empty, so krdc tried to "fix" it by replacing the scheme with vnc:// Once I had that working, I found out that the newly created view was immediately destro