[FFmpeg-devel] [PATCH 4/5] ffbuild/common.mak: clean up and move fftools/resources-specific code its Makefile

[softworkz .]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Ramiro Polla
> Sent: Dienstag, 27. Mai 2025 04:05
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 4/5] ffbuild/common.mak: clean up and move
> fftools/resources-specific code its Makefile
> 
> On Tue, May 27, 2025 at 3:48 AM softworkz .
> <softworkz-at-hotmail.com at ffmpeg.org> wrote:
> > > -----Original Message-----
> > > From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Ramiro
> Polla
> > > Sent: Dienstag, 27. Mai 2025 03:33
> > > To: ffmpeg-devel at ffmpeg.org
> > > Subject: [FFmpeg-devel] [PATCH 4/5] ffbuild/common.mak: clean up and move
> > > fftools/resources-specific code its Makefile
> [...]
> > I actually wanted to push this tomorrow:
> >
> >
> https://patchwork.ffmpeg.org/project/ffmpeg/patch/pull.80.v4.ffstaging.FFmpeg.
> 1748037922309.ffmpegagent at gmail.com/
> >
> > As discussed with Timo, the V4 keeps .ptx files unconditionally while
> > all other intermediates are handled as intermediates per Gnu make
> > default behavior (i.e. get deleted)
> 
> It seems we don't delete any other intermediate files (i.e.: ptx and
> vulkan). At least I don't remember seeing the rm commands at the end
> of builds before like they appear now. I don't think we need to make
> an exception for the resource files.

This is not about making exceptions. The use of .SECONDARY in order to retain
intermediates has only been a workaround because the rebuild condition that
came from the .d files hadn't been understood.
This is fixed now and make can do its job as intended without any workarounds.

Thanks
sw