[FFmpeg-cvslog] r15741 - trunk/doc/general.texi

Michael Niedermayer michaelni
Wed Oct 29 23:57:50 CET 2008


On Wed, Oct 29, 2008 at 11:54:41PM +0100, Diego Biurrun wrote:
> On Wed, Oct 29, 2008 at 10:56:54AM +0100, Michael Niedermayer wrote:
> > On Wed, Oct 29, 2008 at 07:48:59AM +0100, diego wrote:
> > > 
> > > Log:
> > > Add 'make checkheaders' to developer guidelines and patch checklist.
> > 
> > Now besides skiping others oppinions ...
> > 
> > This change is bad because
> > It adds one more point to a already rather long list, and one that is
> > really silly. 
> 
> I keep telling people that their patches need to pass 'make
> checkheaders' during reviews.  Compn pointed out that this should really
> be listed in the policy/checklist.  That was all.
> 

> > If there is a general agreement / majority in favor of running make
> > checkheaders, then we should have a single target in the Makefile that
> > runns all tests (regression & checkheaders) and this should be in the
> > policy & checklist. Its 1 point instead of 2 and otherwise does the
> > same.
> 
> I will add such a target in a moment.

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20081029/08e3f9bd/attachment.pgp>



More information about the ffmpeg-cvslog mailing list