Re: [PATCH] dpkg-base: Permit multiple clean runs

[Henning Schild <henning.schild@siemens.com>]

Am Mon, 16 Sep 2019 08:46:07 +0200
schrieb "[ext] Jan Kiszka" <jan.kiszka@siemens.com>:

> From: Jan Kiszka <jan.kiszka@siemens.com>
> 
> "find ${S}/.." fails if the is no ${S} anymore.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>  meta/classes/dpkg-base.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/dpkg-base.bbclass
> b/meta/classes/dpkg-base.bbclass index c86d0b3..e814208 100644
> --- a/meta/classes/dpkg-base.bbclass
> +++ b/meta/classes/dpkg-base.bbclass
> @@ -102,7 +102,7 @@ addtask dpkg_build before do_build
>  CLEANFUNCS += "repo_clean"
>  
>  repo_clean() {
> -    DEBS=$( find ${S}/.. -maxdepth 1 -name "*.deb" )
> +    DEBS=$( find ${S}/.. -maxdepth 1 -name "*.deb" || true )

"|| true" swallows too many potential errors

go for

[ ! -d ${S} ] && return 0

Henning

>      if [ -n "${DEBS}" ]; then
>          for d in ${DEBS}; do
>              p=$( dpkg-deb --show --showformat '${Package}' ${d} )