Debian Bug report logs - #697221
motion: No longer has support for mysql

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Dick Middleton <dick@lingbrae.com>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: motion: No longer has support for mysql

Date: Wed, 02 Jan 2013 20:34:42 +0000

Package: motion
Version: 3.2.12-3.2
Severity: serious
Justification: 4

Dear Maintainer,

   * What led up to the situation?

upgrade from squeeze to wheezy

   * What exactly did you do (or not do) that was effective (or
     ineffective)?

no longer connects to mysql
  reports unknown config option mysql_*

Jan  2 19:42:42 hagrid motion: [0] Unknown config option "mysql_host"
Jan  2 19:42:42 hagrid motion: [0] Unknown config option "mysql_db"
Jan  2 19:42:42 hagrid motion: [0] Unknown config option "mysql_user"
Jan  2 19:42:42 hagrid motion: [0] Unknown config option "mysql_password"

  ldd doesn't include mysql libs

ldd /usr/bin/motion | grep -i sql

All indicates it's not built with mysql support.


-- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (990, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-4-amd64 (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash

Versions of packages motion depends on:
ii  adduser                3.113+nmu3
ii  debconf [debconf-2.0]  1.5.48
ii  libavcodec53           6:0.8.4-1
ii  libavformat53          6:0.8.4-1
ii  libavutil51            6:0.8.4-1
ii  libc6                  2.13-37
ii  libjpeg8               8d-1
ii  libpq5                 9.1.7-1

Versions of packages motion recommends:
ii  ffmpeg  6:0.8.4-1

Versions of packages motion suggests:
ii  mysql-client                     5.5.28+dfsg-1
ii  mysql-client-5.5 [mysql-client]  5.5.28+dfsg-1
pn  postgresql-client                <none>

-- Configuration Files:
/etc/default/motion changed:
start_motion_daemon=yes

/etc/init.d/motion changed:
NAME=motion
PATH_BIN=/bin:/usr/bin:/sbin:/usr/sbin
DAEMON=/usr/bin/motion
PIDFILE=/var/run/$NAME.pid
DEFAULTS=/etc/default/$NAME
DESC="motion detection daemon"
ENV="env -i LANG=C PATH=$PATH_BIN"
. /lib/lsb/init-functions
test -x $DAEMON || exit 0
RET=0
[ -r "$DEFAULTS" ] && . "$DEFAULTS" || start_motion_daemon=yes
check_daemon_enabled () {
    if [ "$start_motion_daemon" = "yes" ] ; then
        return 0
    else
        log_warning_msg "Not starting $NAME daemon, disabled via /etc/default/$NAME"
        return 1
    fi
}
case "$1" in
  start)
    if check_daemon_enabled ; then
	log_daemon_msg "Starting $DESC" "$NAME" 
if start-stop-daemon --start --oknodo --exec $DAEMON -b --chuid motion ; then
            log_end_msg 0
        else
            log_end_msg 1
            RET=1
        fi
    fi
    ;;
  stop)
    log_daemon_msg "Stopping $DESC" "$NAME"
    if start-stop-daemon --stop --oknodo --exec $DAEMON --retry 30 ; then
        log_end_msg 0
    else
        log_end_msg 1
        RET=1
    fi
    ;;
  reload|force-reload)
    log_daemon_msg "Reloading $NAME configuration"
    if start-stop-daemon --stop --signal HUP --exec $DAEMON ; then
        log_end_msg 0
    else
        log_end_msg 1
        RET=1
    fi
    ;;
  restart-motion)
    if check_daemon_enabled ; then
        log_action_begin_msg "Restarting $NAME"
        if $0 stop && $0 start ; then
            log_action_end_msg 0
        else
            log_action_cont_msg "(failed)"
            RET=1
        fi
    fi
    ;;
  restart)
    $0 restart-motion
    ;;
  *)
    echo "Usage: /etc/init.d/$NAME {start|stop|restart|reload}"
    RET=1
    ;;
esac
exit $RET

/etc/motion/motion.conf changed:
daemon on
quiet on
text_right %d-%b-%y\n%H:%M:%S
target_dir /var/motion
output_normal off
low_cpu 2
post_capture 10
control_html_output on
control_port 8084
control_localhost on  # only localhost can access
snapshot_filename %u.%a.%b%d/cam%t/s%Y%m%d%H%M%S-%v
jpeg_filename %u.%a.%b%d/cam%t/p%H%M%S-%q-%v
on_movie_end /etc/motion/offsiteit %f
sql_log_mpeg on
sql_query insert into security(camera, filename, frame, file_type, time_stamp, event_time_stamp) values('%t', '%f', '%q', '%n', '%Y-%m-%d %T', '%C')
mysql_host localhost
mysql_db motion 
mysql_user web
mysql_password web
webcam_quality 80
webcam_maxrate 10
webcam_motion on
thread /etc/motion/cam-elm.conf
thread /etc/motion/cam-ash.conf
thread /etc/motion/cam-oak.conf
thread /etc/motion/cam-larch.conf

/etc/motion/thread1.conf [Errno 2] No such file or directory: u'/etc/motion/thread1.conf'
/etc/motion/thread2.conf [Errno 2] No such file or directory: u'/etc/motion/thread2.conf'
/etc/motion/thread3.conf [Errno 2] No such file or directory: u'/etc/motion/thread3.conf'
/etc/motion/thread4.conf [Errno 2] No such file or directory: u'/etc/motion/thread4.conf'

-- debconf information:
  motion/moved_conf_dir:

Message #10 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: Dick Middleton <dick@lingbrae.com>, 697221@bugs.debian.org

Subject: Re: Bug#697221: motion: No longer has support for mysql

Date: Thu, 3 Jan 2013 12:17:58 +0100

[Message part 1 (text/plain, inline)]

Hi Dick,

On Wed, Jan 02, 2013 at 08:34:42PM +0000, Dick Middleton wrote:

> upgrade from squeeze to wheezy
> 
>    * What exactly did you do (or not do) that was effective (or
>      ineffective)?
> 
> no longer connects to mysql
>   reports unknown config option mysql_*

The issue here seems to be a broken multiarch-build of 3.2.12-3.2.
Can you please try the package at [1] and report if that fixes the issue 
for you? The diff is attached and at least the build-log seems proper 
now.

Regards
Evgeni

[1] http://dorei.kerker.die-welt.net/~evgeni/tmp/motion_3.2.12-3.3_amd64.deb

-- 
Bruce Schneier can read and understand Perl programs.

[motion_697221.diff (text/x-diff, attachment)]

Message #15 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: Dick Middleton <dick@lingbrae.com>, 697221@bugs.debian.org

Subject: Re: Bug#697221: motion: No longer has support for mysql

Date: Thu, 3 Jan 2013 13:22:21 +0100

Control: tag -1 patch

Hi Dick,

On Thu, Jan 03, 2013 at 11:51:20AM +0000, Dick Middleton wrote:
> Evgeni,
> 
> > On Wed, Jan 02, 2013 at 08:34:42PM +0000, Dick Middleton wrote:
> > 
> >> upgrade from squeeze to wheezy
> >>
> >>    * What exactly did you do (or not do) that was effective (or
> >>      ineffective)?
> >>
> >> no longer connects to mysql
> >>   reports unknown config option mysql_*
> > 
> > The issue here seems to be a broken multiarch-build of 3.2.12-3.2.
> > Can you please try the package at [1] and report if that fixes the issue 
> > for you? The diff is attached and at least the build-log seems proper 
> > now.
> 
> A quick try - it seems to work no errors.
> 
> Thank you very much.  Good of you to respond so quickly.

Thanks for the confirmation, I tagged the bug "patch", as it includes a 
working patch for the issue. Btw, please CC the bug when you reply, that 
way others can follow our communication :)

Regards
Evgeni

-- 
Bruce Schneier can read and understand Perl programs.

Message #22 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: 697221@bugs.debian.org

Subject: motion: diff for NMU version 3.2.12-3.3

Date: Sat, 5 Jan 2013 19:37:25 +0100

[Message part 1 (text/plain, inline)]

tags 697221 + pending
thanks

Dear maintainer,

I've prepared an NMU for motion (versioned as 3.2.12-3.3) and
uploaded it to DELAYED/2. Please feel free to tell me if I
should delay it longer.

Regards.

[motion-3.2.12-3.3-nmu.diff (text/x-diff, attachment)]

Message #29 received at 697221-close@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: 697221-close@bugs.debian.org

Subject: Bug#697221: fixed in motion 3.2.12-3.3

Date: Mon, 07 Jan 2013 19:02:36 +0000

Source: motion
Source-Version: 3.2.12-3.3

We believe that the bug you reported is fixed in the latest version of
motion, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 697221@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Evgeni Golov <evgeni@debian.org> (supplier of updated motion package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Thu, 03 Jan 2013 12:00:38 +0100
Source: motion
Binary: motion
Architecture: source amd64
Version: 3.2.12-3.3
Distribution: unstable
Urgency: low
Maintainer: Juan Angulo Moreno <juan@apuntale.com>
Changed-By: Evgeni Golov <evgeni@debian.org>
Description: 
 motion     - V4L capture program supporting motion detection
Closes: 697221
Changes: 
 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
Checksums-Sha1: 
 71a09aaa49102709cf625df29c0752a2b8276037 1195 motion_3.2.12-3.3.dsc
 d21e473db6946734c061811a21095483d514e3f0 37724 motion_3.2.12-3.3.debian.tar.gz
 8fa64df0c1b1d20b62e6a11893ca7143e6cf3932 265124 motion_3.2.12-3.3_amd64.deb
Checksums-Sha256: 
 d210fdba671941e26b9a64e5eaf36979a7c6a08bcc8eceedd001736d1a4213f2 1195 motion_3.2.12-3.3.dsc
 c0cceb5dc64b48202067a7402ff92247fcf81c1ada9b39fd8f31d04c8653f4eb 37724 motion_3.2.12-3.3.debian.tar.gz
 f1fe98d025d9fd4f6422bb8c5029f08bda3d3a5f0ac2e5874aa19f5015b7684e 265124 motion_3.2.12-3.3_amd64.deb
Files: 
 1c0c4a17e6cf1d25e8a9a1845ac33201 1195 graphics optional motion_3.2.12-3.3.dsc
 24057ff333b2fd7b483788a148ba68fc 37724 graphics optional motion_3.2.12-3.3.debian.tar.gz
 c71ca149e9866a391e8568fda769416e 265124 graphics optional motion_3.2.12-3.3_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iEYEARECAAYFAlDocrwACgkQJKAUGKwVtQwOSwCbBuTZhVPxpDWX3mCGtse/WTko
NbYAn0JoqpfT54lbW78P8YXAbr6Vh2fd
=t590
-----END PGP SIGNATURE-----

Message #34 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Julien Cristau <jcristau@debian.org>

To: Evgeni Golov <evgeni@debian.org>, 697221@bugs.debian.org

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Tue, 8 Jan 2013 22:37:45 +0100

[Message part 1 (text/plain, inline)]

Control: reopen -1

On Sat, Jan  5, 2013 at 19:37:25 +0100, Evgeni Golov wrote:

> tags 697221 + pending
> thanks
> 
> Dear maintainer,
> 
> I've prepared an NMU for motion (versioned as 3.2.12-3.3) and
> uploaded it to DELAYED/2. Please feel free to tell me if I
> should delay it longer.
> 
> Regards.

> 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-01-05 19:25:43.000000000 +0100
> @@ -1,3 +1,14 @@
> +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.

1) since you don't have a versioned build-dep on multiarched
libmysqlclient-dev you don't know that it's there
2) this thing should just use AC_CHECK_LIBS already.

> +    Closes: #697221
> +
> + -- Evgeni Golov <evgeni@debian.org>  Thu, 03 Jan 2013 12:00:38 +0100
> +
>  motion (3.2.12-3.2) unstable; urgency=medium
>  
>    * Non-maintainer upload.

Cheers,
Julien

[signature.asc (application/pgp-signature, inline)]

Message #43 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Salvatore Bonaccorso <carnil@debian.org>

To: Julien Cristau <jcristau@debian.org>, 697221@bugs.debian.org

Cc: Evgeni Golov <evgeni@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sat, 12 Jan 2013 20:44:59 +0100

[Message part 1 (text/plain, inline)]

Hi Julien, Hi Evgeni

On Tue, Jan 08, 2013 at 10:37:45PM +0100, Julien Cristau wrote:
> Control: reopen -1
> 
> On Sat, Jan  5, 2013 at 19:37:25 +0100, Evgeni Golov wrote:
> 
> > tags 697221 + pending
> > thanks
> > 
> > Dear maintainer,
> > 
> > I've prepared an NMU for motion (versioned as 3.2.12-3.3) and
> > uploaded it to DELAYED/2. Please feel free to tell me if I
> > should delay it longer.
> > 
> > Regards.
> 
> > 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-01-05 19:25:43.000000000 +0100
> > @@ -1,3 +1,14 @@
> > +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.
> 
> 1) since you don't have a versioned build-dep on multiarched
> libmysqlclient-dev you don't know that it's there
> 2) this thing should just use AC_CHECK_LIBS already.

How about the attached patch? I tried to implement similar how to
detect the multiarch libavcodec path. Attached is the debdiff against
the current version in testing, the debdiff against current unstable
and the patch 03_mysql_multiarch.patch.

Regards,
Salvatore

[motion_3.2.12-3.2_3.2.12-3.4.debdiff (text/plain, attachment)]

[motion_3.2.12-3.4.debdiff (text/plain, attachment)]

[03_mysql_multiarch.patch (text/x-diff, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #48 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Salvatore Bonaccorso <carnil@debian.org>

To: 697221@bugs.debian.org

Cc: Julien Cristau <jcristau@debian.org>, Evgeni Golov <evgeni@debian.org>

Subject: Re: Bug#697221: motion: Version tested and connects to mysql database

Date: Sat, 12 Jan 2013 21:19:48 +0100

Hi

I tested the resulting package, but I do not have a camera to fully
test the functionality. I only verified that the mysql support is
again present and the motion daemon connects to the mysql server.

So if the patch looks okay also from release team view I can prepare
the NMU (unless Juan wants to do it himself).

Regards,
Salvatore

Message #53 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Julien Cristau <jcristau@debian.org>

To: Salvatore Bonaccorso <carnil@debian.org>

Cc: 697221@bugs.debian.org, Evgeni Golov <evgeni@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sat, 12 Jan 2013 22:54:34 +0100

[Message part 1 (text/plain, inline)]

On Sat, Jan 12, 2013 at 20:44:59 +0100, Salvatore Bonaccorso wrote:

> +--- a/configure.in
> ++++ b/configure.in
> +@@ -551,7 +551,7 @@
> + 	if test "${MYSQL_LIBS}" = "yes"; then
> + 		AC_MSG_CHECKING(autodect mysql libs)
> +         	# Autodetect
> +-		for w in /usr/lib64 /usr/lib /usr/local/lib /usr/mysql /usr/local/mysql /usr/local/mysql/lib /opt /opt/mysql; do
> ++		for w in /usr/lib/$(dpkg-architecture -qDEB_BUILD_MULTIARCH); do
> + 			# check for plain setups
> + 			if test -f $w/libmysqlclient.a -o -f $w/libmysqlclient.so; then
> + 				MYSQL_LIBDIR=$w

*puke*

Cheers,
Julien

[signature.asc (application/pgp-signature, inline)]

Message #58 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Salvatore Bonaccorso <carnil@debian.org>

To: Julien Cristau <jcristau@debian.org>

Cc: 697221@bugs.debian.org, Evgeni Golov <evgeni@debian.org>

Subject: Re: Bug#697221: motion: another attempt for mysql support detection

Date: Sun, 13 Jan 2013 10:59:11 +0100

[Message part 1 (text/plain, inline)]

Hi Julien

On Sat, Jan 12, 2013 at 10:54:34PM +0100, Julien Cristau wrote:
> On Sat, Jan 12, 2013 at 20:44:59 +0100, Salvatore Bonaccorso wrote:
> 
> > +--- a/configure.in
> > ++++ b/configure.in
> > +@@ -551,7 +551,7 @@
> > + 	if test "${MYSQL_LIBS}" = "yes"; then
> > + 		AC_MSG_CHECKING(autodect mysql libs)
> > +         	# Autodetect
> > +-		for w in /usr/lib64 /usr/lib /usr/local/lib /usr/mysql /usr/local/mysql /usr/local/mysql/lib /opt /opt/mysql; do
> > ++		for w in /usr/lib/$(dpkg-architecture -qDEB_BUILD_MULTIARCH); do
> > + 			# check for plain setups
> > + 			if test -f $w/libmysqlclient.a -o -f $w/libmysqlclient.so; then
> > + 				MYSQL_LIBDIR=$w
> 
> *puke*

Okay ...

Would an approach using mysql_config from libmysqlclient-dev package
be a more viable solution? I tried to look around how other solved the
problem.

p.s.: only trying to help on open RC bugs ;-)

Regards,
Salvatore

[motion_3.2.12-3.2_3.2.12-3.4.debdiff (text/plain, attachment)]

[motion_3.2.12-3.4.debdiff (text/plain, attachment)]

Message #63 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: Julien Cristau <jcristau@debian.org>, 697221@bugs.debian.org

Cc: Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sun, 27 Jan 2013 15:22:37 +0100

[Message part 1 (text/plain, inline)]

Hi Julien,

On Tue, Jan 08, 2013 at 10:37:45PM +0100, Julien Cristau wrote:

> 2) this thing should just use AC_CHECK_LIBS already.

how about the atteched one? The debdiff is against -3.2 in testing.
It's pretty aggressive and kills --with-mysql-* settings (thinking of those, I should remove them alltogether then).
Does it look good enough for you?

Regards
Evgeni

[motion_3.2.12-3.4.debdiff (text/plain, attachment)]

Message #68 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Julien Cristau <jcristau@debian.org>

To: Evgeni Golov <evgeni@debian.org>

Cc: 697221@bugs.debian.org, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sun, 27 Jan 2013 15:29:42 +0000

[Message part 1 (text/plain, inline)]

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.

Cheers,
Julien

[signature.asc (application/pgp-signature, inline)]

Message #73 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Evgeni Golov <evgeni@debian.org>

To: Julien Cristau <jcristau@debian.org>

Cc: 697221@bugs.debian.org, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sun, 27 Jan 2013 20:08:55 +0100

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>".

But mysql_config keeps telling me (on my debian machine):
root@nana:/# mysql_config  --include
-I/usr/include/mysql
root@nana:/# mysql_config  --cflags 
-I/usr/include/mysql -DBIG_JOINS=1  -fno-strict-aliasing  -g

greets
Evgeni

-- 
Bruce Schneier can read and understand Perl programs.

Message #78 received at 697221@bugs.debian.org (full text, mbox, reply):

From: gregor herrmann <gregoa@debian.org>

To: Evgeni Golov <evgeni@debian.org>, 697221@bugs.debian.org

Cc: Julien Cristau <jcristau@debian.org>, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sun, 3 Mar 2013 15:17:28 +0100

[Message part 1 (text/plain, inline)]

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

[697221_testing.diff (text/x-diff, attachment)]

[697221_unstable.diff (text/x-diff, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #83 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Michael Gilbert <mgilbert@debian.org>

To: 697221@bugs.debian.org

Subject: re: motion: No longer has support for mysql

Date: Sun, 17 Mar 2013 16:46:55 -0400

control: tag -1 -patch

Removing tag since it looks like theres some more work to do.

Best wishes,
Mike

Message #90 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Jonathan Wiltshire <jmw@debian.org>

To: gregor herrmann <gregoa@debian.org>, 697221@bugs.debian.org

Cc: Evgeni Golov <evgeni@debian.org>, Salvatore Bonaccorso <carnil@debian.org>, Julien Cristau <jcristau@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Sun, 24 Mar 2013 12:39:27 +0000

[Message part 1 (text/plain, inline)]

On Sun, Mar 03, 2013 at 03:17:28PM +0100, gregor herrmann wrote:
> 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 :)

It's been some weeks again since this; are we any closer to a resolution?

From glancing at the options Gregor's mysql_config patch looks reasonable,
though I have not tested it. It is certainly minimal.

Removal is also an option of course, though it would be a shame to waste
your work on this bug. But if motion is to be in Wheezy, it needs to be
maintained for at least the next three years and it currently isn't getting
any love from the maintainer at all.


-- 
Jonathan Wiltshire                                      jmw@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

<directhex> i have six years of solaris sysadmin experience, from
            8->10. i am well qualified to say it is made from bonghits
			layered on top of bonghits

[signature.asc (application/pgp-signature, inline)]

Message #95 received at 697221@bugs.debian.org (full text, mbox, reply):

From: gregor herrmann <gregoa@debian.org>

To: Jonathan Wiltshire <jmw@debian.org>

Cc: 697221@bugs.debian.org, Evgeni Golov <evgeni@debian.org>, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Mon, 25 Mar 2013 18:35:22 +0100

[Message part 1 (text/plain, inline)]

On Sun, 24 Mar 2013 12:39:27 +0000, Jonathan Wiltshire wrote:

> > 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 :)
> 
> It's been some weeks again since this; are we any closer to a resolution?

My guess is that everbody was waiting for an advice by the release
team; so thanks for chiming in :)
 
> From glancing at the options Gregor's mysql_config patch looks reasonable,
> though I have not tested it. It is certainly minimal.

Ok, thanks.

I just found a small issue with it: squeeze's mysql_config doesn't
know about the "--variable=[pkglibdir]", so we technically need a
versioned dependency on libmysqlclient-dev (probably >= 5.5, I don't
find anything in the upstream or Debian changelog).

In this case we might as well go with Evgeni's original patch, and
just add a ">= 5.5.17-4" to the libmysqlclient-dev build dependency.
 
> Removal is also an option of course, though it would be a shame to waste
> your work on this bug. 

More importantly, it would make its users unhappy (which are not so few).

> But if motion is to be in Wheezy, it needs to be
> maintained for at least the next three years and it currently isn't getting
> any love from the maintainer at all.

Right, the package could be maintained a bit more actively; anyway,
the bug should be fixed in unstable in any case, and the removal
question is up to the RT :)


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: Rolling Stones: Midnight Rambler

[signature.asc (application/pgp-signature, inline)]

Message #100 received at 697221@bugs.debian.org (full text, mbox, reply):

From: Jonathan Wiltshire <jmw@debian.org>

To: gregor herrmann <gregoa@debian.org>

Cc: 697221@bugs.debian.org, Evgeni Golov <evgeni@debian.org>, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Mon, 25 Mar 2013 18:45:57 +0000

[Message part 1 (text/plain, inline)]

On Mon, Mar 25, 2013 at 06:35:22PM +0100, gregor herrmann wrote:
> On Sun, 24 Mar 2013 12:39:27 +0000, Jonathan Wiltshire wrote:
> 
> > > 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 :)
> > 
> > It's been some weeks again since this; are we any closer to a resolution?
> 
> My guess is that everbody was waiting for an advice by the release
> team; so thanks for chiming in :)

:)
  
> > From glancing at the options Gregor's mysql_config patch looks reasonable,
> > though I have not tested it. It is certainly minimal.
> 
> Ok, thanks.
> 
> I just found a small issue with it: squeeze's mysql_config doesn't
> know about the "--variable=[pkglibdir]", so we technically need a
> versioned dependency on libmysqlclient-dev (probably >= 5.5, I don't
> find anything in the upstream or Debian changelog).
> 
> In this case we might as well go with Evgeni's original patch, and
> just add a ">= 5.5.17-4" to the libmysqlclient-dev build dependency.

Ok, either of those sounds sensible. Perhaps you could choose one and see
how it gets on with the auto-builders?

Evgeni's original patch got an ack from the reporter at least, so maybe
that's the one to go for.


Cheers,


-- 
Jonathan Wiltshire                                      jmw@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

<directhex> i have six years of solaris sysadmin experience, from
            8->10. i am well qualified to say it is made from bonghits
			layered on top of bonghits

[signature.asc (application/pgp-signature, inline)]

Message #105 received at 697221@bugs.debian.org (full text, mbox, reply):

From: gregor herrmann <gregoa@debian.org>

To: Jonathan Wiltshire <jmw@debian.org>

Cc: 697221@bugs.debian.org, Evgeni Golov <evgeni@debian.org>, Salvatore Bonaccorso <carnil@debian.org>

Subject: Re: Bug#697221: motion: diff for NMU version 3.2.12-3.3

Date: Mon, 25 Mar 2013 20:58:44 +0100

[Message part 1 (text/plain, inline)]

Control: tag -1 + patch pending

On Mon, 25 Mar 2013 18:45:57 +0000, Jonathan Wiltshire wrote:

> > > From glancing at the options Gregor's mysql_config patch looks reasonable,
> > > though I have not tested it. It is certainly minimal.
> > Ok, thanks.
> > 
> > I just found a small issue with it: squeeze's mysql_config doesn't
> > know about the "--variable=[pkglibdir]", so we technically need a
> > versioned dependency on libmysqlclient-dev (probably >= 5.5, I don't
> > find anything in the upstream or Debian changelog).
> > 
> > In this case we might as well go with Evgeni's original patch, and
> > just add a ">= 5.5.17-4" to the libmysqlclient-dev build dependency.
> 
> Ok, either of those sounds sensible. Perhaps you could choose one and see
> how it gets on with the auto-builders?

Ok, in this case ...
 
> Evgeni's original patch got an ack from the reporter at least, so maybe
> that's the one to go for.

... I also prefer Evgeni's original patch with the version added to
the build dependency.

I've now uploaded -3.4 with the attached debdiff to DELAYED/1, in case
there are some further comments.


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: Zueriwest: Prinz

[697221_motion_3.2.12-3.4.diff (text/x-diff, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #112 received at 697221-close@bugs.debian.org (full text, mbox, reply):

From: gregor herrmann <gregoa@debian.org>

To: 697221-close@bugs.debian.org

Subject: Bug#697221: fixed in motion 3.2.12-3.4

Date: Tue, 26 Mar 2013 20:49:11 +0000

Source: motion
Source-Version: 3.2.12-3.4

We believe that the bug you reported is fixed in the latest version of
motion, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 697221@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
gregor herrmann <gregoa@debian.org> (supplier of updated motion package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Mon, 25 Mar 2013 20:52:43 +0100
Source: motion
Binary: motion
Architecture: source amd64
Version: 3.2.12-3.4
Distribution: unstable
Urgency: low
Maintainer: Juan Angulo Moreno <juan@apuntale.com>
Changed-By: gregor herrmann <gregoa@debian.org>
Description: 
 motion     - V4L capture program supporting motion detection
Closes: 697221
Changes: 
 motion (3.2.12-3.4) unstable; urgency=low
 .
   * Non-maintainer upload.
   * Fix "No longer has support for mysql", step 2:
     make build dependency on libmysqlclient-dev versioned;
     5.5.17-4 is the first upload with complete multi-arch support, as needed
     by dpkg-architecture in debian/rules.
     (Closes: #697221)
Checksums-Sha1: 
 3e67ee423f51fb8e6e03f272cf6f8373ab834566 1849 motion_3.2.12-3.4.dsc
 dacbecd59b1a15e93883c6343cbb44f7ff930dae 38022 motion_3.2.12-3.4.debian.tar.gz
 9400b12039a7b9e2225bd1df1c0c16def01ad6d5 265848 motion_3.2.12-3.4_amd64.deb
Checksums-Sha256: 
 020bbaf2a698382265d3f4bfef7db64fdc9a42fc9de83435696eb3b8730a3136 1849 motion_3.2.12-3.4.dsc
 7b1fab15814456672a3f25c63a5a9939a7a77c326580c11b875bb247ad99660e 38022 motion_3.2.12-3.4.debian.tar.gz
 b4b2496527427f3a6913d6f5377b6f8f73b48d9914a5716883ef73ad8fc1425a 265848 motion_3.2.12-3.4_amd64.deb
Files: 
 63d88093f3377a961765caff1526c34b 1849 graphics optional motion_3.2.12-3.4.dsc
 9ca0c29936b23757dbe354ba2e8678fc 38022 graphics optional motion_3.2.12-3.4.debian.tar.gz
 be3e6b66343dc1f2efaa8e18778c3aea 265848 graphics optional motion_3.2.12-3.4_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBCAAGBQJRUKs+AAoJELs6aAGGSaoGGVMQAJyMykI3/HORn+bsuDcV5lbn
4k3bBw9xyr+THxxBLqQ+EN/L/31GR5I3BkzBYXe/suEIbBDe9u3SubEyv8sghEKl
IWIgjUzK1bOuOnp+z1HlZ/clhlfkExkczvVC8b/Tu+hT/bh07wp/ncsdLCmnO1f0
aLJyQS0E0nWNj8QbTcI6xg1VEkOD71hmP9pmgVny5B3sfMEt4+T29izfbPT1/JEM
ojSylXNjd2pJDBfFQ5/waNYKc3twCRo/eUjNFTKDW+L50vW2YkyKV5+z+zmIURyw
fpOpznv55Wjfi+160sqsVdaGki3z2Qpbn/fHR3IGx8jS73Fhnk3LL4tNa+CWh86k
GI/clqKAKFnkRhwevZjlTlrXpdQH7+xU24cjaCMJdzufeJmEn+LfLQXfylBgqdOa
G9zjDkV5iBaI1TDCmhgbedPn3Wf0PVpjVgnGfEJ/ofqCf7JmRinVWkOZHPd58QfY
J8bFmx72B89NnZuGRqzxAq9YIK7qZ5dfLld0L6j1clGUTy/HRo+yvxDjj8fEY6U+
B5cX0RMpZ3C/fLOiMmHbEsylGokk8hoKTtDFhYIpukJEbbZQbQXDF9eoiQcDsT+r
ad0LNwKclH8uZIxiAcT/Jrt2JIYsgdLYJvAGNKwp5Kq8hvZoN1DvDSazYm5MRdUq
iGUnXXIDmlNY8t2+YvXl
=p9P0
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.