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

[andrew]

On Fri, Mar 28, 2008 at 03:08:32PM +0100, Paul TT wrote:
> On Thu, 27 Mar 2008 06:48:32 +0100
> Chidambar 'ilLogict' Zinnoury <illogict at online.fr> wrote:

> >  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.
> 
> yes, %U is correct, following the specs.

I can see my small patch is doing fairly poorly here and will die
shortly :-) Nevertheless a casual google search will show that this is
a widespread problem with gmplayer and files with spaces in their
names or paths. It is seen in Debian and Ubuntu, both of whom are
patching there mplayer releases with %F and my own experience with
Slackware / xfce reveals the same problem.

If the substitution of %F or nothing for %U is not a solution I
believe another more sophisticated solution would be nice to see.

         Andrew

-- 
http://www.andrews-corner.org