[Pkg-ace-devel] Switching to dpatch

[Thomas Girard]

Hi Raphael,

I had not enough time this week to switch the existing packages to
dpatch, and I won't be able to work on this until monday.

However, I had a quick look at how dpatch works, and how you have
modified debian/rules to use it (using sources from [1]).

Here are a few notes about what I've understood:
   1. The .orig.tar.gz contains only the upstream tarball. The .diff.gz
      contains only the debian/ directory and a patches/ subdirectory
      that will be managed by dpatch.
   2. The upstream tarball gets decompressed when building the package,
      thus creating an ACE_wrappers subdirectory, so many debian
      files had to be changed to use this new root for location of
      sources. The debian/rules then applies all the patches from
      patches/
   3. debian/rules was modified to use debian.features, a new file you
      wrote that:
      - is used by MPC to regenerate all GNUmakefile if needed (target
        build-mwc).
      - is used as a base for platform_macros.GNU
      The regeneration of all GNUmakefile by MPC is a very long
      operation, so you have chosen to save the diff of the generated
      GNUmakefiles into 20-reactors-makefiles.dpatch.

If anything here is wrong, please let me know.

Now, I have two questions :
1. Do you know why 20-reactors-makefiles.dpatch is so big (40 Mb) ?
2. Your previous modifications to debian/rules to have make pass
    -DACE_NDEBUG -DACE_USE_RCSID to cpp were removed. Is this
    intentional ?

Also, debian.features has rwho=1, while it is not being built AFAIK.

If my analysis of how you're using dpatch and MPC is not completely
wrong, and if you are busy at the beginning of the next week, I'm still
okay to port these changes myself. Otherwise, you may need this
information (answering your mail with two options):

1.1) I've been using quilt so far to manage patches. Changes to dpatch
is quite straightforward.
1.2) I've made a preliminary conversion to dpatch. They are available at
      http://thomas.g.girard.free.fr/ace/dpatch but need rework. Indeed,
      some of them could be folded, and they need to be renumbered:
      third patch is debian.diff.
1.3) The source used is a slightly modified version of the latest beta
      kit to have correct timestamps. It is available from the Debian
      package page for ACE.


Thanks,

Thomas


[1]
http://einstein.speech-design.de/ftppub/debian-speechdesign/pool/main/a/ace/