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

[Diego Biurrun]

On Thu, Mar 27, 2008 at 01:08:03PM +1100, andrew wrote:
> 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:
> 
> > > --- etc/mplayer.desktop	(revision 26271)
> > > +++ etc/mplayer.desktop	(working copy)
> > > @@ -10,7 +10,7 @@
> > >  TryExec=gmplayer
> > > -Exec=gmplayer %U
> > > +Exec=gmplayer %F
> > 
> > According to the standard these are not the same thing:
> > http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s06.html
> > 
> > %F  A list of files. Use for apps that can open several local files at
> >     once. Each file is passed as a separate argument to the executable
> >     program.
> > 
> > %U  A list of URLs. Each URL is passed as a separate argument to the
> >     executable program. Local files may either be passed as file: URLs
> >     or as file path.
> 
> 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.

I'm still not 100% happy, but I have applied your patch.  The way I
see it, no better solution is going to materialize and this is a minor
and acceptable workaround.  Plus, distributions are already deploying
this.

Diego