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

Jim DeLaHunt from.ffmpeg-dev at jdlh.com
Wed Nov 22 02:35:41 EET 2017


On 2017-11-21 16:05, Carl Eugen Hoyos wrote:
> 2017-11-22 0:59 GMT+01:00 Jim DeLaHunt<from.ffmpeg-dev at jdlh.com>:
>> On 2017-11-21 15:40, Carl Eugen Hoyos wrote:
>>> 2017-11-21 22:41 GMT+01:00 Jim DeLaHunt<from.ffmpeg-dev at jdlh.com>:
>>>
>>>> - at 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.
>>> I am against removing this paragraph.
> [...]
>
>> 2. Instead of /removing/ that paragraph, are you also against replacing it
>> with the same paragraph but centred on the ffmpeg-devel list?  The patch has
>> the following alternative:
>>
>> + at section Documentation/Other
>> + at subheading Subscribe to the ffmpeg-devel mailing list.
>> +It is important to be subscribed to the
>> + at uref{https://lists.ffmpeg.org/mailman/listinfo/ffmpeg-devel, ffmpeg-devel}
>> +mailing list, because any patch you contribute must be sent there
>> +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.
>> +Also, this list is where bugs and possible improvements or
>> +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.
> Sorry, I do not understand this paragraph / question.
>
> If the alternative may lead to another answer than above, please consider
> sending a complete patch.

Carl Eugen:

I believe I sent a complete patch.

Part of what that patch does is replace a section

    "@subheading Subscribe to the ffmpeg-cvslog mailing list."

with a similar section

    "@subheading Subscribe to the ffmpeg-devel mailing list."

Git put the delete lines first, then the replacement lines. See the 
lines labelled, "@@ -381,12 +379,19 @@". Do you see that part of the patch?

Also,
 > [...]

I didn't see you answer my question 1, "Do you expect all contributors 
to ffmpeg to subscribe to the ffmpeg-cvslog email list //in addition 
to// the ffmpeg-devel list?" I think that question is relevant to 
deciding how to handle this @subheading section.
> Thank you, Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Best regards,
       —Jim DeLaHunt

-- 
     --Jim DeLaHunt, jdlh at jdlh.com     http://blog.jdlh.com/ (http://jdlh.com/)
       multilingual websites consultant

       355-1027 Davie St, Vancouver BC V6E 4L2, Canada
          Canada mobile +1-604-376-8953



More information about the ffmpeg-devel mailing list