On Sun, 27 Jan 2013 20:08:55 +0100, Evgeni Golov wrote:

> On Sun, Jan 27, 2013 at 03:29:42PM +0000, Julien Cristau wrote:
> > On Sun, Jan 27, 2013 at 15:22:37 +0100, Evgeni Golov wrote:
> > 
> > > ++         TEMP_CFLAGS="$TEMP_CFLAGS -I/usr/include/mysql"
> > 
> > This bit is probably not quite right in general.  Not sure what the
> > correct way to get the header location is.
> 
> My brain keeps telling me drop that shit and "#include <mysql/mysql.h>".

This bug hasn't seen any activity since over a month; let's get back
on track.

Some thoughts:
- I think fixing this on the packaging side is less invasive than
  hacking the upstream autofoo stuff.
- IMO, Evgeni's original patch in -3.3 matches what has been done for
  quite a few other packages regarding to multiarch; obviously, the
  build dependency on libmysqlclient-dev should be versioned.
- An alternative would be to use mysql_config, as proposed by
  Salvatore; patches against -3.2 and -3.3 attached.
- Salvatore's patch (with "dpkg-architecture -qDEB_BUILD_MULTIARCH")
  in configure.in) indeed looks not so elegant; OTOH that's the same as
  in the already existing debian/patches/02_libav_multiarch.patch, so
  it's at least consistent.

I guess someone needs to decide what we want to have in wheezy :)


Cheers,
gregor

-- 
 .''`.  Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
 : :' : Debian GNU/Linux user, admin, and developer  -  http://www.debian.org/
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: U2: New York
diff -Nru motion-3.2.12/debian/changelog motion-3.2.12/debian/changelog
--- motion-3.2.12/debian/changelog	2012-06-06 12:25:16.000000000 +0200
+++ motion-3.2.12/debian/changelog	2013-03-03 15:07:12.000000000 +0100
@@ -1,3 +1,26 @@
+motion (3.2.12-3.4) UNRELEASED; urgency=low
+
+  * Non-maintainer upload.
+  * Fix "No longer has support for mysql": try a simpler fix than in -3.3:
+    - revert the build-dependency on dpkg-dev in debian/control and the usage
+      of DEB_HOST_MULTIARCH in debian/rules
+    - use mysql_config to find the libddir, and pass it to configure's
+      --with-mysql-dir in debian/rules
+    (Closes: #697221)
+
+ -- gregor herrmann <gre...@debian.org>  Sun, 03 Mar 2013 14:52:52 +0100
+
+motion (3.2.12-3.3) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Properly find MySQL in /usr/lib/$(DEB_HOST_MULTIARCH).
+    + Build-Depend on dpkg-dev >= 1.16
+    + Get DEB_HOST_MULTIARCH from dpkg-architecture.
+    + Pass --with-mysql-dir=/usr/lib/$(DEB_HOST_MULTIARCH) to configure.
+    Closes: #697221
+
+ -- Evgeni Golov <evg...@debian.org>  Thu, 03 Jan 2013 12:00:38 +0100
+
 motion (3.2.12-3.2) unstable; urgency=medium
 
   * Non-maintainer upload.
diff -Nru motion-3.2.12/debian/rules motion-3.2.12/debian/rules
--- motion-3.2.12/debian/rules	2012-06-06 12:24:30.000000000 +0200
+++ motion-3.2.12/debian/rules	2013-03-03 15:11:44.000000000 +0100
@@ -8,6 +8,8 @@
 CFLAGS = -Wall -g
 LDFLAGS = -Wl,--as-needed
 
+MYSQLLIBDIR=$(shell mysql_config --variable=pkglibdir)
+
 ifneq (,$(findstring noopt,$(DEB_BUILD_OPTIONS)))
 	CFLAGS += -O0
 else
@@ -32,6 +34,7 @@
 		--sysconfdir=/etc/motion \
 		--with-ffmpeg \
 		--with-mysql \
+		--with-mysql-lib=$(MYSQLLIBDIR) \
 		--with-pgsql \
 		--without-optimizecpu \
 		--without-jpeg-mmx
diff -Nru motion-3.2.12/debian/changelog motion-3.2.12/debian/changelog
--- motion-3.2.12/debian/changelog	2013-01-05 19:25:43.000000000 +0100
+++ motion-3.2.12/debian/changelog	2013-03-03 15:07:12.000000000 +0100
@@ -1,3 +1,15 @@
+motion (3.2.12-3.4) UNRELEASED; urgency=low
+
+  * Non-maintainer upload.
+  * Fix "No longer has support for mysql": try a simpler fix than in -3.3:
+    - revert the build-dependency on dpkg-dev in debian/control and the usage
+      of DEB_HOST_MULTIARCH in debian/rules
+    - use mysql_config to find the libddir, and pass it to configure's
+      --with-mysql-dir in debian/rules
+    (Closes: #697221)
+
+ -- gregor herrmann <gre...@debian.org>  Sun, 03 Mar 2013 14:52:52 +0100
+
 motion (3.2.12-3.3) unstable; urgency=low
 
   * Non-maintainer upload.
diff -Nru motion-3.2.12/debian/control motion-3.2.12/debian/control
--- motion-3.2.12/debian/control	2013-01-05 19:25:27.000000000 +0100
+++ motion-3.2.12/debian/control	2013-03-03 14:57:40.000000000 +0100
@@ -3,7 +3,7 @@
 Priority: optional
 Maintainer: Juan Angulo Moreno <j...@apuntale.com>
 HomePage: http://motion.sf.net
-Build-Depends: debhelper (>= 6), dpkg-dev (>= 1.16), dh-autoreconf, libjpeg-dev, zlib1g-dev, libavcodec-dev, libavformat-dev, libpq-dev, libmysqlclient-dev, libv4l-dev
+Build-Depends: debhelper (>= 6), dh-autoreconf, libjpeg-dev, zlib1g-dev, libavcodec-dev, libavformat-dev, libpq-dev, libmysqlclient-dev, libv4l-dev
 Standards-Version: 3.9.2
 
 Package: motion
diff -Nru motion-3.2.12/debian/rules motion-3.2.12/debian/rules
--- motion-3.2.12/debian/rules	2013-01-05 19:25:27.000000000 +0100
+++ motion-3.2.12/debian/rules	2013-03-03 15:11:44.000000000 +0100
@@ -3,11 +3,13 @@
 
 DEB_HOST_GNU_TYPE   ?= $(shell dpkg-architecture -qDEB_HOST_GNU_TYPE)
 DEB_BUILD_GNU_TYPE  ?= $(shell dpkg-architecture -qDEB_BUILD_GNU_TYPE)
-DEB_HOST_MULTIARCH  ?= $(shell dpkg-architecture -qDEB_HOST_MULTIARCH)
+
 
 CFLAGS = -Wall -g
 LDFLAGS = -Wl,--as-needed
 
+MYSQLLIBDIR=$(shell mysql_config --variable=pkglibdir)
+
 ifneq (,$(findstring noopt,$(DEB_BUILD_OPTIONS)))
 	CFLAGS += -O0
 else
@@ -32,7 +34,7 @@
 		--sysconfdir=/etc/motion \
 		--with-ffmpeg \
 		--with-mysql \
-		--with-mysql-lib=/usr/lib/$(DEB_HOST_MULTIARCH) \
+		--with-mysql-lib=$(MYSQLLIBDIR) \
 		--with-pgsql \
 		--without-optimizecpu \
 		--without-jpeg-mmx

Attachment: signature.asc
Description: Digital signature

Reply via email to