[FFmpeg-devel] [PATCH v8 0/3] ffbuild/commonmak: Fix rebuild check with implicit rule chains

From: ffmpegagent <ffmpegagent-at-gmail.com@ffmpeg.org>
To: ffmpeg-devel@ffmpeg.org
Cc: softworkz <softworkz@hotmail.com>
Subject: [FFmpeg-devel] [PATCH v8 0/3] ffbuild/commonmak: Fix rebuild check with implicit rule chains
Date: Mon, 23 Jun 2025 22:56:03 +0000
Message-ID: <pull.80.v8.ffstaging.FFmpeg.1750719366.ffmpegagent@gmail.com> (raw)

V2
==

 * Fix MSVC build
   (use the universal command pattern)

V3
==

 * Skip dependency generation by clearing CC_DEPS instead
   (as suggested by Ramiro - thanks!)

V4
==

 * Always keep .ptx files (as suggested by Timo - thanks) Tested all
   scenarios:
   * .ptx.c and .ptx.gz still get deleted (as intermediates)
   * repeated make shows "up-to-date"
   * removing a .ptx file does not cause a rebuild (it's still an
     intermediate, but an "intermediate to keep")
   * but changing a .ptx does (in case of dev/debugging)
   * changed .cu files always rebuild of course

V5
==

 * First patch remains unchanged
 * Added second patch to clean up and consolidate the rules around
   compression

V6
==

 * Rebased
 * Confirmed that it also resolves MSVC-CLang compilation
   (as reported by Kasper Michalow - thanks!)

V7
==

 * As the log line about intermediate file deletion ("RM ....") didn't find
   much love, this version uses the workaround via the .SECONDARY special
   make target to prevent intermediate file deletion

V8
==

 * Resubmit due to Patchwork outage

.

softworkz (3):
  fftools/resources: Fix double-build by disabling .d file generation
  ffbuild/commonmak: Consolidate pattern rules for compression
  fftools/resources: Update .gitignore

 ffbuild/common.mak           | 49 ++++++++++++++----------------------
 fftools/Makefile             |  1 +
 fftools/resources/.gitignore |  5 ++--
 fftools/resources/Makefile   | 12 ++++++---
 4 files changed, 31 insertions(+), 36 deletions(-)

base-commit: e6298e0759430f64e9bd9298775de92597be8a50
Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-80%2Fsoftworkz%2Fsubmit_commonmak-v8
Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-80/softworkz/submit_commonmak-v8
Pull-Request: https://github.com/ffstaging/FFmpeg/pull/80

Range-diff vs v7:

 1:  391217966f = 1:  391217966f fftools/resources: Fix double-build by disabling .d file generation
 2:  17b0af91ec = 2:  17b0af91ec ffbuild/commonmak: Consolidate pattern rules for compression
 3:  dde90b288f = 3:  cfeeade9d4 fftools/resources: Update .gitignore

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