Comment 9 for bug 13458

Jérôme Marant <email address hidden> writes:

> Quoting Kurt Roeckx <email address hidden>:
>
>
>> The problem is this line in your rules file:
>> debian/autofiles.diff: debian/patches/*.dpatch
>> ${update_debian_autofiles_diff}
>>
>> This causes timestamp skew issues because the
>> debian/autofiles.diff file is in the
>> emacs21_21.3+1-9.diff.gz before the debian/patches/*
>> so it thinks it needs to rebuild those.
>>
>> In the emacs21_21.3+1-8.diff.gz file the order of the
>> files was different so it did not show that problem.
>
> Rob, do you have any idea about how to fix this?

The only thing I've thought of so far is to change debian/rules to not
rely on timestamps for this file. Instead keep a file of the md5sums
of the *.dpatch files and use that and cmp to determine when to
re-generate. i.e. (in psudeo-make code):

  define update_autofiles_diff_if_needed
    md5sum debian/*.dpatch > debian/tmp-dpatch.md5
    if ! cmp debian/dpatch.md5 debian/tmp-dpatch.md5; \
    then \
      $(update_debian_autofiles_diff); \
      md5sum debian/*.dpatch > debian/dpatch.md5; \
    fi
  endef

Then I'd just have to arrange for this action to be taken at all the
appropriate times. A bit ugly perhaps, but it looks like it might
work.

--
Rob Browning
rlb @defaultvalue.org and @debian.org; previously @cs.utexas.edu
GPG starting 2002-11-03 = 14DD 432F AE39 534D B592 F9A0 25C8 D377 8C7E 73A4