control: owner -1 !
control: tags -1 moreinfo

Hi,>   * Non-maintainer upload.

changes are huge, but being half mia, and on lowNMU threshold...
(and too many bugs here, so lets do it)

>   * New upstream release (closes: #833081, #811988, #798624, #645616).>   * 
> Updated to Standards-Version 3.9.8 (no changes required).
>   * Updated to debhelper 9, updated Build-Depends.
>   * Added Multiarch support.
>   * Updated debian/*.install and removed the install directory where 
>source and
>     destination directories are the same.
>   * Added patch to generate non-empty index.html in the documentation.
>   * Removed old-style config file.
>   * Commented out the broken Vcs URLs in debian/control.

1) have patches been upstreamed?
2) patch description might be nice / patches should follow dep3 and
 please don't remove the original fields (e.g. author)

3) d/p/003-no-old-style-config.patch

you are using autoreconf and this patch is patching
this means that the patch will touch a file that is autoregenerated.

So, in case please patch

4) automake, libtool, doxygen, dh-autoreconf

doxygen might be needed only for arch:all builds, so you might want to move
it into Build-Depends-Indep
(probably it won't work, since you seem to build documentation also in arch:any 

5) automake, libtool, are them useful?

6) new files
diff -Nru libcgicc-3.2.9/debian/libcgicc-dev.dirs 
--- libcgicc-3.2.9/debian/libcgicc-dev.dirs     1970-01-01 01:00:00.000000000 
+++ libcgicc-3.2.16/debian/libcgicc-dev.dirs    2016-09-08 20:41:26.000000000 
@@ -0,0 +1 @@
diff -Nru libcgicc-3.2.9/debian/libcgicc-dev.install 
--- libcgicc-3.2.9/debian/libcgicc-dev.install  1970-01-01 01:00:00.000000000 
+++ libcgicc-3.2.16/debian/libcgicc-dev.install 2016-09-08 20:41:26.000000000 
@@ -0,0 +1,4 @@


I have to say I wouldn't oppose too much a rename of the binary from
libcgicc5-dev to libcgicc-dev, but this means going in bin-new queue,
and I would take the opportunity on the next SONAME bump.
(there aren't reverse-dependencies)

7) /usr/bin/cgicc-config

this was shipped before, why are you removing it?

8) library changed soname?


not sure what happened, but this seems bad
objdump -p ../  |grep -i SONAME

soname going backward is somewhat bad.
(unless somebody has a better opinion, probably it doesn't mean
too much, but for sure the binary needs to follow the SONAME change).

Please investigate this with upstream, according to their changelog
they did something wrong and tried to recover from it.
(changing SONAME for this seems a good reason, and will integrate
the Ubuntu patch too, at least in some little way, as well
as the -dev package renaming)



Reply via email to