Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- Date: Thu, 27 Mar 2008 06:48:32 +0100
- From: Chidambar 'ilLogict' Zinnoury <illogict@xxxxxxxxx>
- Subject: Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
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@xxxxxxxxx> 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
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ MPlayer-dev-eng mailing list MPlayer-dev-eng@xxxxxxxxxxxx https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
- Follow-Ups:
- References:
- [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- From: andrew
- Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- From: Diego Biurrun
- Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- From: andrew
- [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- Prev by Date: [MPlayer-dev-eng] [PATCH] Added audio format s32le and s32be for pulseaudio output
- Next by Date: Re: [MPlayer-dev-eng] [PATCH] Support for DragonFly BSD
- Previous by thread: Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- Next by thread: Re: [MPlayer-dev-eng] [PATCH] Fix for mplayer desktop file
- Index(es):