Hi,

On Mon, May 08, 2017 at 06:17:19PM -0500, Dirk Eddelbuettel wrote:
>  a) your line was missing the square brackets

Works without.

>  b) when I add it as in
> 
>      PKG_CHECK_MODULES([CPPUNIT], [cppunit >= 1.12.1])
> 
>    it doesn't somehow make it into configure so built still fails

Works for me. I fixed zipios++ in exactly that way:

$ cat debian/patches/pkg-config.diff 
Description: use pkg-config instead of cppunit-config
 cppunit-config (and AM_PATH_CPPUNIT) was removed in cppunit 1.14
Author: Rene Engelhard <r...@debian.org>
Origin: vendor
Forwarded: no
Bug-Debian: https://bugs.debian.org/862126
Last-Update: 2017-05-08

--- zipios++-0.1.5.9+cvs.2007.04.28.orig/configure.ac
+++ zipios++-0.1.5.9+cvs.2007.04.28/configure.ac
@@ -41,7 +41,7 @@ AC_ARG_WITH(
 AC_CHECK_LIB([z], [inflate], , AC_MSG_ERROR([zlib is required to compile 
$PACKAGE.]), [$ZLIB_LDFLAGS])
 AC_CHECK_LIB([z], [zError], AC_DEFINE([HAVE_ZERROR], [1], [Define if zlib has 
zError]), , [$ZLIB_LDFLAGS])
 
-AM_PATH_CPPUNIT(1.6.0)
+PKG_CHECK_MODULES(CPPUNIT, cppunit >= 1.6.0)
 
 # Checks for header files.
 AC_HEADER_DIRENT

Results in

checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for CPPUNIT... yes

Uploaded to experimental.
(Admittedly in 0.1.5.9+cvs.2007.04.28-7 I forgot the pkg-config
build-dependency. Too late here. Fixed in -8)

>  c) the whole bug report was a little annoying as _we don't even have
>     cppunit 1.14 so I am just going to ignore this.

*Shrugs*. I thought of giving advance warning so people can adapt and not
just come over with a (RC) bug later...

And that's why I said that it's available on people.debian.org :)
If it was me who officially maintains cppunit in Debian it would already
be in NEW.
See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=861718

Regards,

Rene

Reply via email to