-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129208/#review100093
-----------------------------------------------------------



I'm not sure it's a very good idea to introduce behavior changes in kdelibs 4. 
Why can't you use kde-open from kf5?

- Aleix Pol Gonzalez


On Oct. 17, 2016, 12:18 p.m., Gerhard Gappmeier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129208/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2016, 12:18 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> RFC3986 defines the following syntax for URI schemes
> scheme = alpha *( alpha | digit | "+" | "-" | "." )
> 
> The '.' was missing which leaded to the problem that
> xdg-open opc.tcp://localhost didn't work.
> 
> Background:
> xdg-open calls kde-open which then uses KUrl and so leaded to this
> problem. Of course this bug may affect also many other locations
> using KUrl::isRelativeUrl.
> 
> This fix also adds a unit test to test the fix and avoid future
> regressions.
> 
> 
> Diffs
> -----
> 
>   kdecore/io/kurl.cpp d9661eab5f8df3bfbbd2900246437fb129037dba 
>   kdecore/tests/kurltest.cpp 3947724bfd1d26ad69b97789205a814a8d712591 
> 
> Diff: https://git.reviewboard.kde.org/r/129208/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gerhard Gappmeier
> 
>

Reply via email to