[FFmpeg-devel] [PATCH 0/5] doc/developer: Add subsection about patch submission via FFstaging
softworkz .
softworkz at hotmail.com
Thu May 8 14:13:31 EEST 2025
> -----Original Message-----
> From: ffmpegagent <ffmpegagent at gmail.com>
> Sent: Donnerstag, 8. Mai 2025 12:22
> To: ffmpeg-devel at ffmpeg.org
> Cc: softworkz <softworkz at hotmail.com>
> Subject: [PATCH 0/5] doc/developer: Add subsection about patch submission via
> FFstaging
>
> Signed-off-by: softworkz softworkz at 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
More information about the ffmpeg-devel
mailing list