[Pkg-bareos-devel] Bug#769096: Bug#769096: bareos-storage: unowned files after purge (policy 6.8, 10.8): /etc/bareos/.rndpwd

[Jörg Steffens]

Hello,

sorry for the late reply.
Thanks for the hints.
I added the proposed purge. The patch is now in Bareos internal testing
and will go to master later today.

regards,
Jörg


Am 16.11.2014 17:38, schrieb Evgeni Golov:
> Hi,
> 
> thanks for the report (and piuparts!)
> 
> On 11/11/2014 11:38 AM, Andreas Beckmann wrote:
> 
>> during a test with piuparts I noticed your package left unowned files on
>> the system after purge, which is a violation of policy 6.8 (or 10.8):
>>
>> http://www.debian.org/doc/debian-policy/ch-maintainerscripts.html#s-removedetails
>>
>> Filing this as important as having a piuparts clean archive is a release
>> goal since lenny.
>>
>> >From the attached log (scroll to the bottom...):
>>
>> 0m32.2s ERROR: FAIL: Package purging left files on system:
>>   /etc/bareos/.rndpwd	 not owned
>>   /root/.rnd	 not owned
>>
>> Don't worry about /root/.rnd, that is an openssl artefact.
> 
> Yeah, openssl is a bit stupid here, one could call it with
> "HOME=$(mktemp -d)", but I do not think it is worth it either.
> 
> For the rndpwd file I propose the following fix:
> 
> diff --git a/debian/bareos-common.postrm b/debian/bareos-common.postrm
> index e69de29..d18dad6 100644
> --- a/debian/bareos-common.postrm
> +++ b/debian/bareos-common.postrm
> @@ -0,0 +1,18 @@
> +#! /bin/sh
> +
> +set -e
> +
> +
> +case "$1" in
> +  purge)
> +    rm -f /etc/bareos/.rndpwd
> +  ;;
> +esac
> +
> +
> +# dh_installdeb will replace this with shell code automatically
> +# generated by other debhelper scripts.
> +
> +#DEBHELPER#
> +
> +exit 0
> 
> Joerg, does this sound good to you?
> 
> Greets
> Evgeni
> 


-- 
 Jörg Steffens                   joerg.steffens at bareos.com
 Bareos GmbH & Co. KG            http://www.bareos.com

 Sitz der Gesellschaft: Köln | Amtsgericht Köln: HRA 29646
 Komplementär: Bareos Verwaltungs-GmbH
 Geschäftsführer:
 S. Dühr, M. Außendorf, Jörg Steffens, P. Storz, M. v. Wieringen