Hi Jean Baptiste,

On Sun, Nov 16, 2014 at 06:12:56PM +0100, Jean Baptiste Favre wrote:
> I had a look on it during Debian BSP in Paris.

Thanks for fixing RC bugs!

> Problem is located into ola-rdm-test.postinst script:
> - It uses debconf, for variable ola-rdm-tests/daemon, without providing
> any template file
> - It uses db_get and never db_input, thus ola-rdm-tests/daemon is never set
> - debconf is not mentioned as dependency
> 
> Please find attached a patch which removes debconf usage from postinst
> script.

I looked into this patch in more detail now and am willing to sponsor
your NMU (which has been approved by Wouter by now). Unfortunately, this
is not the only serious issue in ola-rdm-tests. In order to avoid
multiple round-trips through the release team I advise /not/ to upload
this fix as is (or at least only request an unblock after fixing the
remaining issues).

Your patch enables ola-rdm-tests to be installed. This is great and it
enables us to remove it again. Unfortunately, after removing and purging
/etc/defaults/ola-rdm-tests remains. The package fails to clean up after
itself.

Please try to improve on your patch to cover this case as well. Please
try to use the tool piuparts to verify that this issue and potentially
more problems are indeed fixed by your improved patch. When you submit a
new patch, please Cc me and attach a .debdiff like you did here. If
anything of the above is unclear, please ask.

Helmut


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

Reply via email to