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

[Chidambar 'ilLogict' Zinnoury]

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!

 Chidambar

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080327/e0522395/attachment.pgp>