Re: [FFmpeg-devel] [PATCH 0/5] doc/developer: Add subsection about patch submission via FFstaging

["softworkz ." <softworkz-at-hotmail.com@ffmpeg.org>]

> -----Original Message-----
> From: ffmpegagent <ffmpegagent@gmail.com>
> Sent: Donnerstag, 8. Mai 2025 12:22
> To: ffmpeg-devel@ffmpeg.org
> Cc: softworkz <softworkz@hotmail.com>
> Subject: [PATCH 0/5] doc/developer: Add subsection about patch submission via
> FFstaging
> 
> Signed-off-by: softworkz softworkz@hotmail.com
> 
> softworkz (5):
>   doc/developer: Move checklist into Submitting Patches chapter
>   doc/developer: Move codecs/formats checklist into Development Policy
>     chapter
>   doc/developer: Reorder Submission procedures content
>   doc/developer: Merge Review paragraphs and deduplicate
>   doc/developer: Add subsection about patch submission via FFstaging
> 
>  doc/developer.texi | 234 ++++++++++++++++++++++++---------------------
>  1 file changed, 127 insertions(+), 107 deletions(-)
> 
> 
> base-commit: 1b643e3f65d75a4e6a25986466254bdd4fc1a01a
> Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-
> 76%2Fsoftworkz%2Fsubmit_website_update-v1
> Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-
> 76/softworkz/submit_website_update-v1
> Pull-Request: https://github.com/ffstaging/FFmpeg/pull/76
> --
> ffmpeg-codebot


Forgot so save the cover letter text before submitting:


The https://ffmpeg.org/developer.html (doc/developer.texi) page contains a lot of good information but there's some structural and contentual duplication that is begging for cleanup:

[3 Development Policy](https://ffmpeg.org/developer.html#Development-Policy-1)
[4 Submitting patches](https://ffmpeg.org/developer.html#Submitting-patches-1)
[5 New codecs or formats checklist](https://ffmpeg.org/developer.html#New-codecs-or-formats-checklist)
[6 Patch submission checklist](https://ffmpeg.org/developer.html#Patch-submission-checklist)
[7 Patch review process](https://ffmpeg.org/developer.html#Patch-review-process)

- a Patch submission checklist clearly belongs to the context of  "Submitting Patches"
- a checklist about the requirements for adding new codecs/formats is mostly a matter of policies
- after moving 5 into 3 and 6 into 4,  another duplication can be resolved elegantly as 4 ends with a section about patch reviews which is then adjacent to the current 7 (patch review process)
- inside 4:
  - some duplication between 4's intro and the checklist can be removed
  - the second part becomes "Submission procedures" with three subsections (send-email, format-patch and FFstaging) and the content separated in a more logical way


Thanks
sw
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".