Re: [FFmpeg-devel] [PATCH 0/3] Clean up build spam from graph css builder

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

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Derek
> Buitenhuis
> Sent: Dienstag, 20. Mai 2025 20:28
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH 0/3] Clean up build spam from graph css builder
> 
> I am not sure how this made it in.
> 
> Before:
>     $ rm fftools/resources/graph.css.c
>     $ make
>     # Must start with a tab in the real Makefile
>     sed 's!/\\*.*\\*/!!g' fftools/resources/graph.css \
>     | tr '\n' ' ' \
>     | tr -s ' ' \
>     | sed 's/^ //; s/ $//' \
>     > fftools/resources/graph.css.min
>     GEN     fftools/resources/graph.css.min.gz
>     BIN2C   fftools/resources/graph.css.c
>     CC      fftools/resources/graph.css.o
>     LD      ffmpeg_g
>     STRIP   ffmpeg
>     LD      ffprobe_g
>     STRIP   ffprobe
>     rm fftools/resources/graph.css.min.gz fftools/resources/graph.css.min
>     $
> 
> After:
>     $ rm fftools/resources/graph.css.c
>     $ make
>     GEN     fftools/resources/graph.css.min
>     GEN     fftools/resources/graph.css.min.gz
>     BIN2C   fftools/resources/graph.css.c
>     CC      fftools/resources/graph.css.o
>     LD      ffmpeg_g
>     STRIP   ffmpeg
>     LD      ffprobe_g
>     STRIP   ffprobe
>     $
> 
> Derek Buitenhuis (3):
>   ffbuild/common: Remove what appears to be a temporary debugging
>     comment
>   ffbuild/common: Properly tag/suppress sed command
>   fftools/resoirces: Mark .css.min and .css.min.gz as NOTINTERMEDIATE
> 
>  ffbuild/common.mak         | 3 +--
>  fftools/resources/Makefile | 1 +
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> --

Hi Derek,

thanks a lot for the patch. This partially duplicates what Timo had
already submitted:

https://patchwork.ffmpeg.org/project/ffmpeg/patch/20250516230202.355445-1-timo@rothenpieler.org/

Regarding patch 3/3, would you mind taking a look at the patch that I 
have submitted in this regard, from which I believe that it's the "most
correct" way:

https://patchwork.ffmpeg.org/project/ffmpeg/patch/pull.80.v2.ffstaging.FFmpeg.1747549830700.ffmpegagent@gmail.com/

Thanks again,
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".