From b461f0bce248ef36aea9b869826a329311648870 Mon Sep 17 00:00:00 2001 From: Georgios Jason Efstathiou Date: Mon, 5 Oct 2026 15:31:17 +0200 Subject: [PATCH] ryph: round source mtimes down instead of to the commit time On an incremental image build, setting a changed file to the commit time can make it older than its stale build output, so cargo would skip the rebuild. Rounding down to the whole second still survives the image layer and never moves a file back past its outputs. --- .ryph/setup.yml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.ryph/setup.yml b/.ryph/setup.yml index 3bf5e00b6..9a8fe2546 100644 --- a/.ryph/setup.yml +++ b/.ryph/setup.yml @@ -1,11 +1,11 @@ steps: # An image layer keeps whole-second mtimes, but cargo fingerprints the - # nanoseconds it saw at build time, so every crate with a build script - # rebuilt on every run. Building from whole-second mtimes keeps the warm - # target/ fresh. Image builds only: at run time this would hide the files - # the run's checkout changed. - - name: pin source mtimes to the commit time (image builds only) - run: '[ "$RYPH_SETUP_PHASE" != image ] || git ls-files -z | xargs -0 touch -d "@$(git log -1 --format=%ct)"' + # nanoseconds it saw at build time. Rounding tracked files down to the + # second before building keeps the warm target/ fresh at run time. Never + # set them to the commit time: on an incremental image build that can make + # a changed file older than its stale build output. + - name: round source mtimes down to whole seconds (image builds only) + run: '[ "$RYPH_SETUP_PHASE" != image ] || git ls-files -z | xargs -0 perl -e ''for (@ARGV) { next if -l; my $m = (lstat)[9]; utime $m, $m, $_ }''' - name: install npm dependencies run: npm install --prefer-offline - name: install Playwright WebKit