[FFmpeg-devel] [PATCH/RFC]Change development policy

[Michael Niedermayer]

On Tue, Oct 05, 2010 at 12:42:45AM +0200, Stefano Sabatini wrote:
> On date Tuesday 2010-10-05 00:24:31 +0200, Michael Niedermayer encoded:
> > On Mon, Oct 04, 2010 at 11:45:28PM +0200, Stefano Sabatini wrote:
> > > On date Monday 2010-10-04 23:28:01 +0200, Benjamin Larsson encoded:
> > > > On 10/04/2010 01:04 PM, Michael Niedermayer wrote:
> > > > > On Mon, Oct 04, 2010 at 11:22:51AM +0200, Carl Eugen Hoyos wrote:
> > > > >> Hi!
> > > > >>
> > > > >> I believe this was originally suggested by Reimar, and since supported by 
> > > > >> several developers.
> > > > >>
> > > > >> Since this may be controversial, please comment in any case and suggest 
> > > > >> wording improvements, flames (including whose idea it was) are less welcome.
> > > > >>
> > > > >> I will consider applying if I receive no comments at all, Carl Eugen
> > > > > 
> > > > >>  developer.texi |    3 +++
> > > > >>  1 file changed, 3 insertions(+)
> > > > >> 290cbac2de815e5b43f4e3fcc065bdc9b13648e4  patchAPI.diff
> > > > >> Index: doc/developer.texi
> > > > >> ===================================================================
> > > > >> --- doc/developer.texi	(revision 25330)
> > > > >> +++ doc/developer.texi	(working copy)
> > > > >> @@ -155,6 +155,9 @@
> > > > >>  
> > > > >>     Note: Redundant code can be removed.
> > > > >>  @item
> > > > >> +   Do not apply patches that change public API without discussing them
> > > > >> +   on the mailing list first.
> > > > >> + at item
> > > > > 
> > > > > I suggest:
> > > > > diff --git a/doc/developer.texi b/doc/developer.texi
> > > > > index e362eec..9c43123 100644
> > > > > --- a/doc/developer.texi
> > > > > +++ b/doc/developer.texi
> > > > > @@ -149,7 +149,8 @@ should also be avoided if they don't make the code easier to understand.
> > > > >     Also if you have doubts about splitting or not splitting, do not hesitate to
> > > > >     ask/discuss it on the developer mailing list.
> > > > >  @item
> > > > > -   Do not change behavior of the program (renaming options etc) without
> > > > > +   Do not change behavior of the program (renaming options,
> > > > > +   changing public API or ABI, etc) without
> > > > >     first discussing it on the ffmpeg-devel mailing list. Do not remove
> > > > >     functionality from the code. Just improve!
> > > > > 
> > > > 
> > > > Both ok but I prefer this one.
> > > 
> > > I prefer the second variant but in the present form is not accurate.
> > > 
> > > API change doesn't imply change in the programs behavior, so I
> > > suggest:
> > > 
> > > Do not change behavior of the programs (renaming options, adding
> > > options, changing the use of pre-existent options, etc) or the public
> > > API or ABI without...
> > 
> > that are a lot of unrelated changes.
> 
> Do not change behavior of the program (renaming options, etc) or
> public API or ABI without ...

yes thats fine with me.
making it programs is of course too but its seperate
and maybe it should be API/ABI of the libraries

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

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101005/0feb4b5b/attachment.pgp>