[FFmpeg-devel] [PATCH 2/3] ffbuild: correctly silence and tag new css/html steps

[softworkz .]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Timo
> Rothenpieler
> Sent: Samstag, 17. Mai 2025 01:02
> To: ffmpeg-devel at ffmpeg.org
> Cc: Timo Rothenpieler <timo at rothenpieler.org>
> Subject: [FFmpeg-devel] [PATCH 2/3] ffbuild: correctly silence and tag new
> css/html steps
> 
> ---
>  ffbuild/common.mak | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/ffbuild/common.mak b/ffbuild/common.mak
> index 4a3cc0c748..31ed19c141 100644
> --- a/ffbuild/common.mak
> +++ b/ffbuild/common.mak
> @@ -140,9 +140,9 @@ else
>  endif
> 
>  # 1) Preprocess CSS to a minified version
> +%.css.min: TAG = SED
>  %.css.min: %.css
> -	# Must start with a tab in the real Makefile
> -	sed 's!/\\*.*\\*/!!g' $< \
> +	$(M)sed 's!/\\*.*\\*/!!g' $< \
>  	| tr '\n' ' ' \
>  	| tr -s ' ' \
>  	| sed 's/^ //; s/ $$//' \
> @@ -151,6 +151,7 @@ endif
>  ifdef CONFIG_RESOURCE_COMPRESSION
> 
>  # 2) Gzip the minified CSS
> +%.css.min.gz: TAG = GZIP
>  %.css.min.gz: %.css.min
>  	$(M)gzip -nc9 $< > $@
> 
> @@ -159,6 +160,7 @@ ifdef CONFIG_RESOURCE_COMPRESSION
>  	$(BIN2C) $< $@ $(subst .,_,$(basename $(notdir $@)))
> 
>  # 4) Gzip the HTML file (no minification needed)
> +%.html.gz: TAG = GZIP
>  %.html.gz: %.html
>  	$(M)gzip -nc9 $< > $@
> 
> --

Thanks for the patch. I had those tags in earlier revisions, dropped them
while investigating the OOT build issue and then forgotten to re-add.

LGTM