Re: [meta-ti] [RFC][master/scarthgap][PATCH 4/4] cc33xx-target-scripts: add cc33xx scripts to control TI cc33xx WLAN device

[Denys Dmytriyenko <denis@denix.org>]

On Thu, Jun 20, 2024 at 03:37:12PM -0500, Khan, Sabeeh wrote:
> 
> On 6/20/2024 3:27 PM, Denys Dmytriyenko wrote:
> >On Wed, Jun 19, 2024 at 10:22:02PM -0500, Sabeeh Khan via lists.yoctoproject.org wrote:
> >>[...]
> >>
> >>diff --git a/meta-ti-extras/recipes-connectivity/cc33xx-target-scripts/cc33xx-target-scripts.bb b/meta-ti-extras/recipes-connectivity/cc33xx-target-scripts/cc33xx-target-scripts.bb
> >>new file mode 100644
> >>index 00000000..2f13f911
> >>--- /dev/null
> >>+++ b/meta-ti-extras/recipes-connectivity/cc33xx-target-scripts/cc33xx-target-scripts.bb
> >>@@ -0,0 +1,22 @@
> >>+DESCRIPTION = "Scripts and configuration files for TI cc33xx wireless drivers"
> >SUMMARY
> 
> Apologies for not being familiar, but are you suggesting here
> 
> to have "SUMMARY" variable instead of "DESCRIPTION", or in addition to?
> 
> Can I please have some more context over the ask here?
> 
> I don't see "SUMMARY" variable in other meta-ti recipes, so I'm
> looking for a reference.

https://docs.yoctoproject.org/ref-manual/variables.html#term-SUMMARY
https://docs.yoctoproject.org/ref-manual/variables.html#term-DESCRIPTION

SUMMARY is a shorter version of the package description (one-line under 72 
characters), while DESCRIPTION can be a longer multi-line description.

DESCRIPTION can be omitted and will be derived from SUMMARY, but not the other 
way around.

In the very old days only the DESCRIPTION was used, hence old recipes still 
don't set SUMMARY, but newly added recipes should not do that.

Here's one of the many recipes in meta-ti that properly uses/sets both:
https://git.yoctoproject.org/meta-ti/tree/meta-ti-extras/recipes-ti/jailhouse/ti-jailhouse.inc


> >>[...]
> >>+
> >>+do_install() {
> >>+	install -d ${D}${datadir}/cc33xx/
> >>+
> >>+	scripts=`find ./* -type f -name "*.*"`
> >Is this find command necessary?
> 
> The recipe should install .sh and .conf files from the git repo only.
> 
> This was done to ignore any README or license files and not have
> them installed.

I believe you can use globs directly with install, unless there's a 
directory hierarchy:

install -m 0755 *.* ${D}${datadir}/cc33xx/

-- 
Denys