[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#835253: RFS: steghide/0.5.1-11 [QA]



Hi,




>This is actually because of lack of dependency on doxygen.
>When you do not have doxygen installed, the manpages are simply not
>created with
>no errors whatsoever.
>Although when you have it there is some difference, it drops to 6 lines,
>instead

>of 300.

-> Considering build-dep doxygen
-> Trying doxygen
-> Installing  debhelper libmcrypt-dev libmhash-dev zlib1g-dev libtool libtool-bin libjpeg-dev dh-autoreconf doxygen


I'm installing it

dpkg -c steghide_0.5.1-11_amd64.deb  |grep man |wc -l
308



no, you can't pollute the usr/share/man/man3 namespace with all the functions of your library.
-rw-r--r-- root/root       275 2016-08-22 23:03 ./usr/share/man/man3/WavFormatChunk.h.3.gz
-rw-r--r-- root/root       869 2016-08-22 23:03 ./usr/share/man/man3/WavPCMSampleValue.3.gz
-rw-r--r-- root/root       291 2016-08-22 23:03 ./usr/share/man/man3/WavPCMSampleValue.h.3.gz
-rw-r--r-- root/root       808 2016-08-22 23:03 ./usr/share/man/man3/WavPCMSampleValueTest.3.gz
-rw-r--r-- root/root       271 2016-08-22 23:03 ./usr/share/man/man3/WavPCMSampleValueTest.h.3.gz
-rw-r--r-- root/root       385 2016-08-22 23:03 ./usr/share/man/man3/WavPCMSmaller.3.gz
-rw-r--r-- root/root       849 2016-08-22 23:03 ./usr/share/man/man3/common.h.3.gz
-rw-r--r-- root/root       383 2016-08-22 23:03 ./usr/share/man/man3/error.h.3.gz
-rw-r--r-- root/root       517 2016-08-22 23:03 ./usr/share/man/man3/gettext.h.3.gz
-rw-r--r-- root/root       304 2016-08-22 23:03 ./usr/share/man/man3/msg.h.3.gz
-rw-r--r-- root/root       374 2016-08-22 23:03 ./usr/share/man/man3/utcommon.h.3.gz
-rw-r--r-- root/root       206 2016-08-22 23:03 ./usr/share/man/man3/wrapper_hash_map.h.3.gz
-rw-r--r-- root/root       205 2016-08-22 23:03 ./usr/share/man/man3/wrapper_hash_set.h.3.gz


this is something that should be installed as a doxygen documentation, somewhere else.

>Ok, this might be something I messed up when removing usage of mkinstalldirs
>script.


wonderful
>Yes, there is a grammar correction for po/de.po indeed.
>Now for the spec file I kind of ignored it intentionally.
>I will review them all.


probably the spec file is autogenerated, not sure

>Right, I was not sure about this.
>At first the dh_clean broke badly so I cleaned it up manually then did this.
>I will fix it.


usually it breaks when you kill the configure script, you have to manually
do: fakeroot debian/rules clean, and maybe delete files created when configure
fails, e.g. config.log

I hope this helps,

G.


Reply to: