#454: 2 patches to 5193 fixes debian package builder (Fixed)

Aug 22 2008 * 18:28
Reported by: Release: 1.2
Priority: Normal Milestone: 1.2.0.2
Component: other Assigned to:

Because of two faults in changeset 5193 branches/rel/1.2, the dpkg-buildpackage -b fails to build debian packages for callweaver. The two patches below allow the command to succeed.

This includes a fix mentioned in another ticket to callweaver-zaptel.install.

I thought that a change to the callweaver.install or related to debian/rules to cause the package builder to simply not look for the /usr/bin directory might be better, but I wasn’t sure that useable but currently de-selected ./configure option might need it. So I just make sure the directory is there, if empty.

--- 1.2/debian/rules    2008-08-22 10:36:41.000000000 -0500
+++ debian-rules-patch1    2008-08-22 10:57:47.000000000 -0500
@@ -125,6 +125,7 @@

     # Add here commands to install the package into debian/tmp.
     $(MAKE) DESTDIR=$(CURDIR)/debian/tmp install
+    mkdir $(CURDIR)/debian/tmp/usr/bin
 #    sed -e "s,__OPENPBX_SBIN_DIR__,/usr/sbin," contrib/scripts/safe_callweaver > \
 #            debian/tmp/usr/sbin/safe_callweaver

--- 1.2/debian/callweaver-zaptel.install    2008-08-22 10:36:41.000000000 -0500
+++ debian-callweaver-zaptell.install-hcpatch1    2008-08-22 10:32:30.000000000 -0500
@@ -1,2 +1,3 @@
 usr/lib*/callweaver/modules/chan_zap.so
-etc/callweaver/zapata.conf
+etc/callweaver/chan_dahdi.conf
+

Changelog:

Modified by Sep 12 2008 * 08:29

Just to confirm that these changes worked fine for me. I downloaded release 1.2 from the repository and had to apply this patch in order to be able to build the Debian package.

Modified by Sep 12 2008 * 20:53
  • Status: changed from Open to Fixed

thanks harry, it’s in changeset 5196