Re: [meta-arago][master/kirkstone][PATCH] mstpd: Add FILES variable

[Denys Dmytriyenko <denis@denix.org>]

On Sat, Aug 12, 2023 at 08:16:54AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> Bring this recipe into compliance with the latest standards.  Add
> missing FILES and update hardcoded /sbin path to $base_sbindir.

Well, you don't have to touch FILES, as the defaults here seem to cover 
everything just fine. Plus, you are adding to the defaults, not replacing 
them, so that's a no-op.

Changing hardcoded /sbin to ${base_sbindir} is all you need here.


> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-arago-extras/recipes-support/mstpctl/mstpd_git.bb | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-extras/recipes-support/mstpctl/mstpd_git.bb b/meta-arago-extras/recipes-support/mstpctl/mstpd_git.bb
> index a410f584..be75b1c5 100644
> --- a/meta-arago-extras/recipes-support/mstpctl/mstpd_git.bb
> +++ b/meta-arago-extras/recipes-support/mstpctl/mstpd_git.bb
> @@ -17,7 +17,15 @@ RDEPENDS:${PN} = "python3-core"
>  
>  inherit autotools
>  
> -EXTRA_OECONF = "--sbindir=/sbin"
> +EXTRA_OECONF = "--sbindir=${base_sbindir}"
> +
> +FILES:${PN} += " \
> +    ${base_sbindir}/mstp* \
> +    ${base_sbindir}/bridge-stp \
> +    ${libexecdir}/mstpctl-utils/* \
> +    ${sysconfdir}/bridge-stp.conf \
> +    ${sysconfdir}/bash_completion.d/mstpctl \
> +"

So, the defaults already have ${base_sbindir}/* ${libexecdir}/* and 
${sysconfdir} and will scoop up everything the recipe installs in those 
directories.

https://git.openembedded.org/openembedded-core/tree/meta/conf/bitbake.conf#n331


>  do_configure:prepend() {
>  	 sed -i -e "s,#!.*python.*,#!${USRBINPATH}/env python3," ${S}/utils/ifquery
> -- 
> 2.17.1