[MPlayer-cvslog] CVS: main/TOOLS midentify,1.3,1.4

Dominik 'Rathann' Mierzejewski dominik at rangers.eu.org
Thu Jun 30 11:59:51 CEST 2005


On Thursday, 30 June 2005 at 02:46, Tobias Diedrich CVS wrote:
> CVS change done by Tobias Diedrich CVS
> 
> Update of /cvsroot/mplayer/main/TOOLS
> In directory mail:/var2/tmp/cvs-serv32709
> 
> Modified Files:
> 	midentify 
> Log Message:
> Better approach to shell escaping, may not catch all cases yet
[...]
>  mplayer -vo null -ao null -frames 0 -identify "$@" 2>/dev/null |
> -	grep "^ID" |
> -	sed -e 's/[`\\!$"]/\\&/g' |
> -	sed -e '/^ID_FILENAME/ { s/^ID_FILENAME=\(.*\)/ID_FILENAME="\1"/g; }'
> +	sed -ne '/^ID_/ {s/[]`\\!$" []/\\&/g;p }'

Is this portable to all sed flavours we should support?

R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"




More information about the MPlayer-cvslog mailing list