Re: [PATCH] dpkg-raw: Fix rebuilding recipes

[Jan Kiszka <jan.kiszka@siemens.com>]

On 09.10.18 11:06, Harald Seiler wrote:
> Rebuilding dpkg-raw packages fails because the
> DEBIAN directories' ownership is transferred to
> root in dpkg_runbuild.  This commit ensures the
> ownership is reset before attempting to write
> metadata.
> 
> Signed-off-by: Harald Seiler <hws@denx.de>
> ---
>   meta/classes/dpkg-raw.bbclass | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/classes/dpkg-raw.bbclass b/meta/classes/dpkg-raw.bbclass
> index c848f3d..806a2d4 100644
> --- a/meta/classes/dpkg-raw.bbclass
> +++ b/meta/classes/dpkg-raw.bbclass
> @@ -18,7 +18,8 @@ addtask install after do_unpack before do_prepare_build
>   
>   deb_package_prepare() {
>   	sudo rm -rf ${D}/DEBIAN
> -	mkdir -p ${D}/DEBIAN
> +	sudo mkdir -p ${D}/DEBIAN
> +	sudo chown ${USER}:${USER} ${D}/DEBIAN

That makes no sense to me: We remove that folder first, and then create it under 
the name of the current user. These two changes are redundant in that light.

Can you describe in more details what you did and how the error messages looked 
like? I frequently rebuild dpkg-raw packages and never saw such an issue recently.

Jan

>   	cat<<-__EOF__ > ${D}/DEBIAN/control
>   		Package: ${PN}
>   		Architecture: ${DISTRO_ARCH}
> 

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux