Re: [OE-core][PATCH 5/5] lib/oe/package_manager: don't try to rm /var/lib/opkg

["Peter Kjellerstedt" <peter.kjellerstedt@axis.com>]

> -----Original Message-----
> From: openembedded-core@lists.openembedded.org <openembedded-
> core@lists.openembedded.org> On Behalf Of Jan Luebbe
> Sent: den 1 april 2020 16:58
> To: openembedded-core@lists.openembedded.org
> Cc: Jan Luebbe <jlu@pengutronix.de>
> Subject: [OE-core][PATCH 5/5] lib/oe/package_manager: don't try to rm
> /var/lib/opkg
> 
> As opkglibdir starts with a /, os.path.join will ignore
> self.target_rootfs, leading to an attempt to remove /var/lib/opkg.
> 
> This only fails if it exists on the host, explaining why this remained
> undiscovered for long.
> 
> Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
> ---
>  meta/lib/oe/package_manager.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/meta/lib/oe/package_manager.py
> b/meta/lib/oe/package_manager.py
> index 2a5d7ef539e6..b0660411eaf5 100644
> --- a/meta/lib/oe/package_manager.py
> +++ b/meta/lib/oe/package_manager.py
> @@ -1796,8 +1796,7 @@ class DpkgPM(OpkgDpkgPM):
>              open(os.path.join(target_dpkg_dir, "available"), "w+").close()
> 
>      def remove_packaging_data(self):
> -        bb.utils.remove(os.path.join(self.target_rootfs,
> -                                     self.d.getVar('opkglibdir')), True)
> +        bb.utils.remove(self.target_rootfs + self.d.getVar('opkglibdir'), True)

Or you could change it to use oe.path.join() instead:

        bb.utils.remove(oe.path.join(self.target_rootfs,
                                     self.d.getVar('opkglibdir')), True)


>          bb.utils.remove(self.target_rootfs + "/var/lib/dpkg/", True)
> 
>      def fix_broken_dependencies(self):
> --
> 2.26.0.rc2

//Peter