[FFmpeg-devel] [PATCH v8 0/4] print_graphs: Complete Filtergraph Printing

[Soft Works]

> -----Original Message-----
> From: Stefano Sabatini <stefasab at gmail.com>
> Sent: Freitag, 21. März 2025 19:05
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Cc: softworkz <softworkz at hotmail.com>
> Subject: Re: [FFmpeg-devel] [PATCH v8 0/4] print_graphs: Complete
> Filtergraph Printing
> 
> On date Wednesday 2025-03-19 17:04:09 +0000, ffmpegagent wrote:
> > Due to the additional work on graph visualization (see
> > https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2025-
> March/341296.html), I
> > have removed the two graph printing commits from this patchset rather
> than
> > adding to it, as to avoid wasting the effort already spent on
> reviewing.
> >
> > So, now it's a graph printing patchset without graph printing :-)
> >
> > All other commits are unchanged from the previous version. I'm also
> > resubmitting to give it a final run of the tests on various platforms.
> >
> > Thanks
> >
> > softworkz (4):
> >   fftools/textformat: Extract and generalize textformat api from
> >     ffprobe.c
> >   fftools/ffprobe: Change to use textformat api
> >   fftools/ffprobe: Rename writer_print_section_* and WriterContext
> >   fftools/ffprobe: Rename AVTextFormatContext variables (w => tfc)
> 
> I'm fine with the changes assuming it passes fate.

It's all green on Patchwork and on the GitHub CI, as well as locally.

Michael had reported a file-not-found compilation error with an x86-32 target. It appears to be an issue with ccache and the fact that there's a new subfolder (which rarely happens).
I used the same configure he gave me (except no ccache) and couldn't reproduce.

> I don't know if you have push commit rights, 

I was going to ask Michael about it, but for now it would be great when you would be able to push it.

Thanks
sw