Re: [FFmpeg-devel] Graphprint Patches Overview

[Kieran Kunhya via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

On Wed, 21 May 2025, 01:45 softworkz ., <softworkz-at-hotmail.com@ffmpeg.org>
wrote:

> Hello,
>
> thanks again to all for the patches. I figured it might be a bit difficult
> to
> keep track of what has already been submitted and fixed and is still
> pending, and I'm sorry that there has been some duplicate effort to fix the
> same things - so here's an overview. The ones with X are the ones I would
> like to apply eventually:
>
>
> Timo Rothenpieler
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14528
> (I would favor "ffbuild/commonmak" over for 1/3)
>
>   [1/3] fftools/resources: fix preservation of intermediary resman build
> artifacts
> X [2/3] ffbuild: correctly silence and tag new css/html steps
> X [3/3] fftools/resources: add missing extensions to .gitignore
>
>
>
> Mark Thompson (already merged)
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14537
>
> X [1/3] ffmpeg: Don't print graphs if there are no graphs to print
> X [2/3] fftools/graphprint: Fix leak of graphprint object
> X [3/3] fftools/graphprint: Fix leak of graph section header string
>
>
> softworkz
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14563
>
> X [1/5] fftools/makefile: Remove resources from ffprobe
> X [2/5] fftools/resources: Use .SECONDARY in Makefile comment
> X [3/5] fftools/ffmpeg: Free print_graph option variables
> X [4/5] fftools/graphprint: Fix memory leaks
> X [5/5] fftools/tf_mermaid: Add missing uninit and fix leaks
>
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14570
> X [v3] ffbuild/commonmak: Fix rebuild check with implicit rule chains
>
>
> Derek Buitenhuis
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=14569
> (1/3 and 2/3 correspond to 2/3 from Timo, and 3/3 doesn't fix
> the rebuild check like "commonmak" above does)
>
> [1/3] ffbuild/common: Remove what appears to be a temporary debugging
> comment
> [2/3] ffbuild/common: Properly tag/suppress sed command
> [3/3] fftools/resoirces: Mark .css.min and .css.min.gz as NOTINTERMEDIATE
>
>
> Thanks again,
> sw
>

Can we just revert the whole set until it's cleaned up properly?

There are more patches to fix issues than the set itself. This is
understandable if it's a bit architectural change like threading but it's
not.

Kieran

>
_______________________________________________
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".