[MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file

[Paul TT]

On Thu, 27 Mar 2008 06:48:32 +0100
Chidambar 'ilLogict' Zinnoury <illogict at online.fr> wrote:

> Dans son message intitulé « Re: [MPlayer-dev-eng] [PATCH] Fix for
> mplayer desktop file » du Thu, 27 Mar 2008 13:08:03 +1100, andrew
> <andrew.david.45 at gmail.com> nous a donné l'occasion de lire :
> 
> > On Thu, Mar 27, 2008 at 02:19:22AM +0100, Diego Biurrun wrote:
> > > On Thu, Mar 27, 2008 at 11:45:23AM +1100, andrew wrote:
> > 
> > Although %F opens urls quite successfully on my system I note the
> > 'optional' part from the same page:
> > 
> > "The Exec key must contain a command line. A command line consists
> > of an executable program optionally followed by one or more
> > arguments."
> > 
> > I found that omission of both %F and %U made no difference on my
> > system and I have changed the patch to reflect this.
> > 
> >    Andrew Strong 
> > 
> 
>  Hello!
> 
>  Putting no argument in the Exec line and expecting a file to be
> opened is not correct. Most file managers will still add the file to
> the command but this behaviour is not to be expected (from 
> http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s06.html
> "If the application should not open any file the %f, %u, %F and %U
> field codes must be removed from the command line and ignored.").
>  It seems that the file manager you are using is not correctly
> escaping the file name before adding it to the command line,
> therefore IMHO there should be a bug in your file manager.
> 
>  Cheers!

yes, %U is correct, following the specs.