[Mplayer-cvslog] CVS: main/DOCS/tech release-howto.txt,1.1,1.2

The Wanderer inverseparadox at comcast.net
Sat Jul 17 09:46:41 CEST 2004


Diego Biurrun CVS wrote:

> +- Ask the DOCS maintainers to commit their final changes, check if
> +  all docs are up to date etc.

Missing comma - "up to date, etc.". "etc." is standing in for further
items in the list, which would be set off with a comma.

> +- update the ChangeLog file (according to CVS log), ask other developers
> +  to verify their parts etc. Ask Diego to spellcheck it.

Another missing comma with "etc."; also, "Update" not capitalized.

> +- Consult at -dev-eng about unstable parts of the code which should be
> +  disabled for the release.
>  
>  create the release tree:
>  - cvs checkout the mplayer src tree
> -- if releasing from 'main', checkout libavcodec from ffmpeg cvs
> -- remove teh CVS subdirs: 
> -    find 0_90 -name CVS -exec rm -rf {} \;
> +- if releasing from 'main', checkout libavcodec from FFmpeg CVS

Shouldn't "checkout" be "check out"? The only argument I can think of
against it is that "checkout" is an actual CVS command, and personally I
don't think that that's enough of a reason, but others might disagree.

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

A government exists to serve its citizens, not to control them.




More information about the MPlayer-cvslog mailing list