[vlc-devel] [PATCH] cli: fixup URLs passed via command-line

Rémi Denis-Courmont remi at remlab.net
Mon Oct 12 16:36:44 CEST 2020


Le maanantaina 12. lokakuuta 2020, 16.52.09 EEST Romain Vimont a écrit :
> On Mon, Oct 12, 2020 at 04:21:13PM +0300, Rémi Denis-Courmont wrote:
> > It was already pointed out multiple times by multiple people that this
> > doesn't work in the previous thread.
> Hmm, that works for me.
> 
> Without the patch, opening an URL containing spaces did not work:
> 
>     ./vlc 'http://someserver/file with space.mp4'
> 
> With this patch, it works.

That seems like a bug. The CLI is expected to be conservative and strict and 
throw errors on invalid parameters. And if we're going to compare with URL 
handlers, curl does not do any weird magic in this case.

But that's moot. As pointed out multiple times in the previous thread, this 
patch breaks tons of MRLs.

-- 
雷米‧德尼-库尔蒙
http://www.remlab.net/





More information about the vlc-devel mailing list