Package: expect-dev
Version: 5.43.0-7
Severity: grave
Justification: renders package unusable
Tags: patch

The static library included in expect-dev has different names
on different architectures. On alpha, mips and mipsel the
library is /usr/lib/libexpect5.43.a, while on the others it's
/usr/lib/libexpect543.a. As the package doesn't include a .so symlink
at all (which I just filed another bug about), this makes the package
unusable since it can't reliably be linked against.

The problem is related to autoconf2.13 #7314: when run under fakeroot,
the configure script fails to detect long file support correctly. This
affects the library filename that is chosen.  When run under sudo (as is
the case on the alpha, mips and mipsel buildds), the long file support
is correctly detected.

The reason why configure gets run with fakeroot is a bug in
debian/rules. The prerequisites of 'config.status' are 'configure',
which is erroneously marked as a phony target, and 'patch', which is not
a regular file. These make the 'config.status' commands get run every
time the 'build' target is invoked, ie. both for 'debian/rules build'
and 'debian/rules binary'.  The latter is run with fakeroot or sudo.

I'm attaching a patch that fixes both debian/rules and the configure test,
to be on the safe side. It might be a good idea to rerun autoconf2.13
first, though, as the configure.in modifications in 08-tcl-stubs.dpatch
aren't currently taking any effect.

Cheers,
-- 
Niko Tyni       [EMAIL PROTECTED]
Index: expect-5.43.0/configure
===================================================================
--- expect-5.43.0.orig/configure        2006-09-30 21:40:44.000000000 +0300
+++ expect-5.43.0/configure     2006-09-30 21:41:07.000000000 +0300
@@ -6342,7 +6342,7 @@
 else
   ac_tmpdirs='/tmp /var/tmp /usr/tmp'
 fi
-for ac_dir in  . $ac_tmpdirs `eval echo $prefix/lib $exec_prefix/lib` ; do
+for ac_dir in  . $ac_tmpdirs ; do
   test -d $ac_dir || continue
   test -w $ac_dir || continue # It is less confusing to not echo anything here.
   (echo 1 > $ac_dir/conftest9012345) 2>/dev/null
Index: expect-5.43.0/debian/rules
===================================================================
--- expect-5.43.0.orig/debian/rules     2006-09-30 21:40:44.000000000 +0300
+++ expect-5.43.0/debian/rules  2006-09-30 21:40:53.000000000 +0300
@@ -18,7 +18,7 @@
        CFLAGS += -O2
 endif
 
-config.status: configure patch
+config.status: configure patch-stamp
        dh_testdir
 
        # Configuring package
@@ -107,4 +107,4 @@
        dh_builddeb
 
 binary: binary-indep binary-arch
-.PHONY: build clean binary-indep binary-arch binary install configure
+.PHONY: build clean binary-indep binary-arch binary install

Reply via email to