> -----Original Message-----
> From: ffmpegagent <ffmpegag...@gmail.com>
> Sent: Donnerstag, 8. Mai 2025 12:22
> To: ffmpeg-devel@ffmpeg.org
> Cc: softworkz <softwo...@hotmail.com>
> Subject: [PATCH 0/5] doc/developer: Add subsection about patch submission via
> FFstaging
> 
> Signed-off-by: softworkz softwo...@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-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to