Re: [PATCH] systemd.bbclass: add RMINITDIR for nativesdk builds

[Trevor Gamblin <trevor.gamblin@windriver.com>]

On 9/27/19 11:07 AM, Trevor Gamblin wrote:

> From: Trevor Gamblin <trevor.gamblin@windriver.com>
>
> nativesdk builds such as nativesdk-dnf are installing systemd
> service files, leading bitbake to throw installed-vs-shipped
> warnings, but these are not needed in nativesdk:
>
>   WARNING: nativesdk-dnf-4.2.2-r0 do_package: QA Issue: nativesdk-dnf: Files/directories were installed but not shipped in any package:
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic.timer
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-makecache.timer
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic.service
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-makecache.service
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-notifyonly.timer
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-download.service
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-download.timer
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-notifyonly.service
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-install.timer
>        /usr/local/oecore-x86_64/sysroots/x86_64-oesdk-linux/lib/systemd/system/dnf-automatic-install.service
>      Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
>      nativesdk-dnf: 13 installed and not shipped files. [installed-vs-shipped]
>
> Rather than have each recipe remove the files, an RMINITDIR case
> for nativesdk builds should be added to systemd.bbclass.
>
> Signed-off-by: Trevor Gamblin <trevor.gamblin@windriver.com>
> ---
>   meta/classes/systemd.bbclass | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta/classes/systemd.bbclass b/meta/classes/systemd.bbclass
> index 747055b8fa..1dca09964b 100644
> --- a/meta/classes/systemd.bbclass
> +++ b/meta/classes/systemd.bbclass
> @@ -231,5 +231,6 @@ python rm_sysvinit_initddir (){
>   
>   do_install[postfuncs] += "${RMINITDIR} "
>   RMINITDIR_class-target = " rm_sysvinit_initddir rm_systemd_unitdir "
> +RMINITDIR_class-nativesdk = " rm_sysvinit_initddir rm_systemd_unitdir "
>   RMINITDIR = ""
>   
Forgot to put you guys on the original cc list. I tested this out with 
systemd and sysvinit in DISTRO_FEATURES with nativesdk-dnf because I was 
having issues with nativesdk-clang building. Switching between the two I 
did not see a rebuild of anything, so I think we're good to go.