Bug#511044: clean-patched depends on configure-stamp

[Jack Bates]

Package: cyrus-imapd-2.2
Version: 2.2.13
Severity: minor
Tags: patch

I encountered this error when trying to build the cyrus-imapd-2.2 package with pbuilder:

checking sasl/sasl.h usability... no
checking sasl/sasl.h presence... no
checking for sasl/sasl.h... no
configure: error: Cannot continue without libsasl2.
Get it from ftp://ftp.andrew.cmu.edu/pub/cyrus-mail/.
make: *** [configure-stamp] Error 1
dpkg-buildpackage: failure: fakeroot debian/rules clean gave error exit status 2

I think this is because pbuilder runs the debian/rules clean target outside of the build root, but the build dependencies are not satisfied outside of the build root. With many packages, I do not encounter this error because the debian/rules clean target does not depend on (most of) the build dependencies.

I dropped the debian/rules clean-patched dependency on configure-stamp and built the package successfully. I wonder if this dependency is really necessary? If it could be safely dropped, then building the package with pbuilder would be more convenient...

-- System Information:
Debian Release: 5.0
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.26-1-686 (SMP w/2 CPU cores)
Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
-------------- next part --------------
--- a/cyrus-imapd-2.2-2.2.13/debian/rules	2009-01-06 16:14:26.000000000 -0800
+++ b/cyrus-imapd-2.2-2.2.13/debian/rules	2009-01-06 16:17:12.000000000 -0800
@@ -92,7 +92,7 @@
 # This is done to make sure the build daemon's source tree is the 
 # same as the one I am using to produce the i386 debs. 
 clean: clean-patched unpatch
-clean-patched: configure-stamp
+clean-patched:
 	dh_testdir
 	dh_testroot
 	-xargs -t -r chmod +x < debian/executable.files