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

[Ivo]

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