[FFmpeg-devel] [PATCH] Refactor Developer Docs, update dev list section (v2)

[Carl Eugen Hoyos]

2017-11-26 9:31 GMT+01:00 Jim DeLaHunt <from.ffmpeg-dev at jdlh.com>:

> - at subsection Documentation/Other
> + at section Documentation/Other
> + at subheading Subscribe to the ffmpeg-devel mailing list.
> +It is important to be subscribed to the

Of course it is important but I would much, much prefer
if people send their patches without being subscribed
than not sending their patches because it is implied
that they cannot send patches if they don't want to
subscribe.

> + at uref{https://lists.ffmpeg.org/mailman/listinfo/ffmpeg-devel, ffmpeg-devel}
> +mailing list, because any patch you contribute must be sent there

No:
I believe it is very important that trivial patches are not sent
to the development mailing list - its volume is already so big
that some patches are sadly (!) forgotten.

> +and reviewed by the other developers. They may have comments about your
> +contribution. We expect you see those comments, and to improve your contribution
> +if requested.

Yes.

But if people are not interested in improving their contribution,
I would still prefer the patches to be sent.

> +Also, this list is where bugs and possible improvements or

I believe this is misleading or even wrong.

> +general questions regarding commits are discussed. That may be helpful
> +information as you write your contribution. Finally, by being a list
> +subscriber your contribution will be posted immediately to the list,
> +without the moderation hold which messages from non-subscribers experience.
> +
>  @subheading Subscribe to the ffmpeg-cvslog mailing list.
> -It is important to do this as the diffs of all commits are sent there and
> -reviewed by all the other developers. Bugs and possible improvements or
> -general questions regarding commits are discussed there. We expect you to
> -react if problems with your code are uncovered.
> +Diffs of all commits are sent to the
> + at uref{https://lists.ffmpeg.org/mailman/listinfo/ffmpeg-cvslog, ffmpeg-cvslog}
> +mailing list. Some developers read this list to review all code base changes
> +from all sources. Subscribing to this list is not mandatory, if
> +all you want to do is submit a patch here and there.

I am (still) against this change.

Sorry, Carl Eugen