Proposed Cyrus 2.3.13 branch (could be: "Re: [SVN] r768")

Henrique de Moraes Holschuh hmh at debian.org
Wed Feb 11 21:48:22 UTC 2009


On Mon, 09 Feb 2009, Duncan Gibb wrote:
> 1) I believe the choice of Berkeley DB version should be made at build
> time, rather than svn commit time.  To this end, I have changed
> debian/control so Cyrus build-depends on libdb4.4-dev | libdb4.5-dev |
> libdb4.6-dev and put the following mess into debian/rules
> 
>   export BDB_VERSION ?= $(shell dpkg-query -l 'libdb4.[2-6]-dev' | grep
> ^ii | sed -e 's|.*\s\libdb\(4\.[0-9]\)-dev\s.*|\1|')

Non-deterministic builds in face of random factors (like a screwed up
chroot or whatever) are never a good idea.

Please revert this part of the change.  Pick whatever version of BDB
is the most stable across the main arches (x86, amd64, ppc...), which
often means asking the BDB maintainer about it, and go with that.
When in doubt, don't use anything openldap isn't using yet.

> 2) I allow epochs in the package name (we use them internally)
> 
> https://mail.incase.de/viewcvs/branches/cyrus23/cyrus-imapd-2.3-2.3.13/debian/cyrus-common-2.3.cyrus2.3.init?r1=767&r2=768

Good.

> 3) I took out the huge and incomprehensible 99-update-autoconf.dpatch in
> favour of having debian/rules run autoconf and autoheader at build time.

That's good, but use autoreconf instead, if possible.

Also, autotooling at build time requires build-depending on a
particular version of each of the required autofoo stuff, AND often
build-conflicting with the old crufty ones.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh



More information about the Pkg-Cyrus-imapd-Debian-devel mailing list