[MPlayer-dev-eng] [PATCH?] [RFC] Script to check "sanity" of MPlayer sources

Ivo ivop at euronet.nl
Sun Aug 7 00:35:02 CEST 2005


On Saturday 06 August 2005 23:42, Torinthiel wrote:
> On Sat, Aug 06, 2005 at 09:05:32PM +0200, Ivo wrote:
> > Updated version:
>
> I have a slightly better/worse method for all_filenames:
> Better, because it catches exactly the files in repository, worse
> because it needs a checkout (i.e. it depends on CVS subdirectories)
>
> all_filenames()
> { ListCVS . }
>
> ListCVS()
> {
>   for f in `grep '^/' $1/CVS/Entries|cut -d/ -f2`; do
>     echo $1/$f
>   done
>   for k in `grep '^D/' $1/CVS/Entries|cut -d/ -f2`; do
>     ListCVS $1/$k
>   done
> }

Nice idea. I'm thinking of having both implementations and an extra command 
line switch. Probably make cvs default, but it will still be possible to 
run the script on an extracted release-tarball by using the -nocvs option.

> And I've successfully used for f in `all_filenames`; do ... instead of
> while read... - the first is IMHO more readable.

Agreed.

Thanks for the feedback!

--Ivo




More information about the MPlayer-dev-eng mailing list