Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-07 Thread gregor herrmann
On Fri, 06 Jan 2012 19:57:04 +, peter green wrote:

 As expected, it still builds for me, and not only on i386 but also on
 amd64.
 Peter, do you have any ideas?
 Not really, when I prepared the patch I think I did so on amd64 and
 the build failed on the amd64 buildd. 

Right, and it still builds for me on amd64.

 All I can think is something
 else changed in between the time
 I prepared the patch and the time the buildds built it.

Or something in the build environment?
* I've looked throught logs a bit and saw a different gcc/g++ version
  on hurd-i386.
* More promising: in my i386 sid cowbuilder chroot, the build
  succeeds with binutils-gold installed and fails without it.  
 
 I may (or may not) take another look at this in a day or so.

Thanks!


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: Funny van Dannen: Arbeiterkinderdenkmal


signature.asc
Description: Digital signature


Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-07 Thread peter green



Or something in the build environment?
* I've looked throught logs a bit and saw a different gcc/g++ version
  on hurd-i386.
* More promising: in my i386 sid cowbuilder chroot, the build
  succeeds with binutils-gold installed and fails without it.
  
heh it's usually the other way round ;) I guess I must have had 
binutils-gold

installed when I was prepping the patch.

The problem seems to be wrong ordering of arguments on the linker 
command line.
It seems that with the link options this package uses and with regular 
ld that if a static
library needs symbols provided by another libary* then the libary that 
needs the

symbols must be before the libary providing them on the linker command line.

A patch reordering the linking is attatched, please add it to the quilt 
series. I can
confirm that with this patch the package builds with both regular 
binutils and

binutils.gold.

*Unfortunately static libraries unlike dynamic libraries do not have an 
internal dependency

list :(
Index: libdecodeqr-0.9.3/src/sample/simple/Makefile.in
===
--- libdecodeqr-0.9.3.orig/src/sample/simple/Makefile.in	2012-01-08 00:09:37.0 +
+++ libdecodeqr-0.9.3/src/sample/simple/Makefile.in	2012-01-08 00:09:37.0 +
@@ -20,7 +20,7 @@
 
 LIBCV=@LIBCV@
 LIBHIGHGUI=@LIBHIGHGUI@
-LIBS=$(LIBHIGHGUI) -lcxcore -ldecodeqr $(LIBCV)
+LIBS=-ldecodeqr $(LIBHIGHGUI) -lcxcore $(LIBCV)
 
 LDFLAGS=@LDFLAGS@ -L$(LIBDECODEQR_DIR)
 
Index: libdecodeqr-0.9.3/src/sample/webcam/Makefile.in
===
--- libdecodeqr-0.9.3.orig/src/sample/webcam/Makefile.in	2012-01-08 00:09:37.0 +
+++ libdecodeqr-0.9.3/src/sample/webcam/Makefile.in	2012-01-08 00:11:41.0 +
@@ -20,7 +20,7 @@
 
 LIBCV=@LIBCV@
 LIBHIGHGUI=@LIBHIGHGUI@
-LIBS=$(LIBHIGHGUI) -lcxcore -ldecodeqr $(LIBCV)
+LIBS=-ldecodeqr $(LIBHIGHGUI) -lcxcore $(LIBCV)
 
 LDFLAGS=@LDFLAGS@ -L$(LIBDECODEQR_DIR)
 


Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-07 Thread gregor herrmann
On Sun, 08 Jan 2012 00:19:17 +, peter green wrote:

 * More promising: in my i386 sid cowbuilder chroot, the build
   succeeds with binutils-gold installed and fails without it.
 heh it's usually the other way round ;) I guess I must have had
 binutils-gold installed when I was prepping the patch.

Yup, me too, in order to work on the usual FTBFS with binutils-gold
or --no-add-needed bugs :)
 
 A patch reordering the linking is attatched, please add it to the
 quilt series. I can confirm that with this patch the package builds
 with both regular binutils and binutils.gold.

Thanks a lot!
I really appreciate all your bug work in general and this quick fix
in particular.
 
Uploaded (after building it with and without -gold too :))

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: Supertramp: Listen To Me Please


signature.asc
Description: Digital signature


Processed: Re: Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-06 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 reopen 652854
Bug #652854 {Done: gregor herrmann gre...@debian.org} [libdecodeqr] 
libdecodeqr FTBFS in unstable configure: error: libcv.la not found.
'reopen' may be inappropriate when a bug has been closed with a version;
you may need to use 'found' to remove fixed versions.
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
652854: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=652854
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-06 Thread gregor herrmann
reopen 652854
thanks

On Wed, 04 Jan 2012 16:15:35 +0100, gregor herrmann wrote:

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

Hm, not the Most Successful NMU Ever, the package FTBFS on almost all
architectures:

https://buildd.debian.org/status/package.php?p=libdecodeqr

As expected, it still builds for me, and not only on i386 but also on
amd64.

Peter, do you have any ideas?

Cheers,
gregor
 
-- 
 .''`.   Homepage: http://info.comodo.priv.at/ - OpenPGP key ID: 0x8649AA06
 : :' :  Debian GNU/Linux user, admin,  developer - http://www.debian.org/
 `. `'   Member of VIBE!AT  SPI, fellow of Free Software Foundation Europe
   `-NP: Elvis Presley: Love Me Tender


signature.asc
Description: Digital signature


Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-06 Thread peter green

gregor herrmann wrote:

reopen 652854
thanks

On Wed, 04 Jan 2012 16:15:35 +0100, gregor herrmann wrote:

  

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



Hm, not the Most Successful NMU Ever, the package FTBFS on almost all
architectures:

https://buildd.debian.org/status/package.php?p=libdecodeqr

As expected, it still builds for me, and not only on i386 but also on
amd64.

Peter, do you have any ideas?
  
Not really, when I prepared the patch I think I did so on amd64 and the 
build failed on the amd64 buildd. All I can think is something else 
changed in between the time

I prepared the patch and the time the buildds built it.

I may (or may not) take another look at this in a day or so.



--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#652854: libdecodeqr: diff for NMU version 0.9.3-6.1

2012-01-04 Thread gregor herrmann
tags 652854 + pending
thanks

Dear maintainer,

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

Regards.

-- 
 .''`.   Homepage: http://info.comodo.priv.at/ - OpenPGP key ID: 0x8649AA06
 : :' :  Debian GNU/Linux user, admin,  developer - http://www.debian.org/
 `. `'   Member of VIBE!AT  SPI, fellow of Free Software Foundation Europe
   `-NP: Rolling Stones: Key-to-the-highway
diff -Nru libdecodeqr-0.9.3/debian/changelog libdecodeqr-0.9.3/debian/changelog
--- libdecodeqr-0.9.3/debian/changelog	2011-09-16 01:18:38.0 +0200
+++ libdecodeqr-0.9.3/debian/changelog	2012-01-04 16:08:01.0 +0100
@@ -1,3 +1,13 @@
+libdecodeqr (0.9.3-6.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fix libdecodeqr FTBFS in unstable configure: error: libcv.la not
+found.: apply patch from peter green that adjusts debian/{control,rules}
+to the restructured opencv libraries.
+Closes: #652854
+
+ -- gregor herrmann gre...@debian.org  Wed, 04 Jan 2012 16:07:17 +0100
+
 libdecodeqr (0.9.3-6) unstable; urgency=low
 
   * Fix FTBFS with ld --as-needed. (Closes: #641565)
diff -Nru libdecodeqr-0.9.3/debian/control libdecodeqr-0.9.3/debian/control
--- libdecodeqr-0.9.3/debian/control	2011-09-16 01:18:38.0 +0200
+++ libdecodeqr-0.9.3/debian/control	2012-01-04 16:01:18.0 +0100
@@ -1,7 +1,7 @@
 Source: libdecodeqr
 Priority: optional
 Maintainer: Nobuhiro Iwamatsu iwama...@debian.org
-Build-Depends: debhelper (= 8), libhighgui-dev, quilt
+Build-Depends: debhelper (= 8), libhighgui-dev, quilt, libcv-dev
 Standards-Version: 3.9.2
 Section: libs
 Homepage: http://trac.koka-in.org/libdecodeqr
diff -Nru libdecodeqr-0.9.3/debian/rules libdecodeqr-0.9.3/debian/rules
--- libdecodeqr-0.9.3/debian/rules	2011-09-16 01:22:25.0 +0200
+++ libdecodeqr-0.9.3/debian/rules	2012-01-04 16:02:12.0 +0100
@@ -5,7 +5,7 @@
 CPPFLAGS+= -I/usr/include/opencv
 LDFLAGS+=-Wl,-z,defs -Wl,--as-needed
 
-export CFLAGS LDFLAGS
+export CPPFLAGS LDFLAGS
 
 EXAMPLEPKG = libdecodeqr-examples
 
@@ -16,11 +16,14 @@
 	- $(MAKE) -C src distclean
 	rm -rf src/Makefile src/libdecodeqr/Makefile src/sample/Makefile src/sample/simple/Makefile src/sample/webcam/Makefile src/test/Makefile
 	rm -rf modules/python/src2/hdr_parser.pyc
-
+	rm -f src/config.log
 	dh_clean
 
 override_dh_auto_configure:
-	dh_auto_configure --sourcedirectory=src
+	dh_auto_configure --sourcedirectory=src -- --with-cv=opencv_core --with-highgui=opencv_highgui 
+	sed -i s/cxcore/opencv_imgproc/ src/libdecodeqr/Makefile
+	sed -i s/cxcore/opencv_imgproc/ src/sample/simple/Makefile
+	sed -i s/cxcore/opencv_imgproc/ src/sample/webcam/Makefile
 
 override_dh_auto_build:
 	$(MAKE) -C src DESTDIR=$(CURDIR)/debian/tmp


signature.asc
Description: Digital signature