On Sat, Apr 16, 2011 at 10:11:59AM -0400, Justin Ruggles wrote:
> On 04/16/2011 07:32 AM, Luca Barbato wrote:
> 
> > --- a/src/contact
> > +++ b/src/contact
> > @@ -17,8 +17,9 @@ corporate disclaimers that claim the content of your 
> > mails is confidential.
> >  Attached patches should not have application/octet-stream as mime-type, but
> >  text/plain or even better text/x-diff or text/x-patch.
> >  If you send us a patch, please make it some form of unified diff. We prefer
> > -patches made using <code>git diff &gt; submission.diff</code> from the 
> > root of
> > -your Libav checkout. For more detailed guidelines, have a look at the
> > +patches made using <code>git format-patch</code> from your Libav local 
> > clone,
> > +they can be sent directly using <code>git send-email</code>.
> > +For more detailed guidelines, have a look at the
> >  <a href="http://www.mplayerhq.hu/DOCS/tech/patches.txt";>MPlayer patch 
> > guidelines</a>
> >  most of what is written there applies to Libav as well.
> >  Trespass these rules at the risk of being flamed.
> 
> The changes look fine,

Fine with me as well, Luca, push it out.

> but I think sometime soon we should add our own list of patch
> guidelines instead of just referring to MPlayer's. There are enough
> differences between the projects that I could see it confusing
> potential contributors.

+1

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to