[meta-arago,master/kirkstone,v2,2/2] ltp-ddt: Add support for version 20230127

The internal testing team would like to move to this version as the
basis for our internal ltp-ddt.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
v2: Add missing space on :append line.

 .../ltp-ddt/ltp-ddt_20230127.bb               | 47 +++++++++++++++++++
 1 file changed, 47 insertions(+)
 create mode 100644 meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb

On Sat, Apr 22, 2023 at 05:07:22PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> The internal testing team would like to move to this version as the
> basis for our internal ltp-ddt.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
> v2: Add missing space on :append line.
> 
>  .../ltp-ddt/ltp-ddt_20230127.bb               | 47 +++++++++++++++++++
>  1 file changed, 47 insertions(+)
>  create mode 100644 meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
> 
> diff --git a/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb b/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
> new file mode 100644
> index 00000000..10b93a87
> --- /dev/null
> +++ b/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
> @@ -0,0 +1,47 @@
> +FILESEXTRAPATHS:prepend := "${COREBASE}/meta/recipes-extended/ltp/ltp:"
> +FILESEXTRAPATHS:prepend := "${ARAGOBASE}/meta-arago-extras/recipes-extended/ltp/ltp:"
> +
> +require recipes-extended/ltp/ltp_${PV}.bb
> +
> +SUMMARY = "Embedded Linux Device Driver Tests based on Linux Test Project"
> +HOMEPAGE = "https://git.ti.com/cgit/test-automation/ltp-ddt/"
> +
> +PROVIDES += "ltp"
> +
> +DEPENDS += "alsa-lib"
> +
> +PE = "1"
> +PR = "r7"
> +PV:append = "+git${SRCPV}"
> +
> +SRCREV = "7f2fc4b7c972f2f188836c739a7175fad2f2b288"
> +BRANCH ?= "master"
> +
> +SRC_URI:remove = "git://github.com/linux-test-project/ltp.git;branch=master;protocol=https"
> +SRC_URI:prepend = "git://git.ti.com/git/test-automation/ltp-ddt.git;protocol=https;branch=${BRANCH} "
> +
> +SRC_URI:append = "https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/plain/include/uapi/linux/rpmsg_rpc.h?h=ti-linux-5.10.y;name=rpmsg_rpc;subdir=linux;downloadfilename=rpmsg_rpc.h "

Gulp. While both :prepend and :append need to ensure whitespaces are 
explicitly added to separate entries in the list, those are needed at 
different places. It is tailing space for :prepend and leading space 
for :append, of course. Or use += here to be safe...


> +SRC_URI[rpmsg_rpc.sha256sum] = "cd237f40a37520a1f2df19fbfeefd00c0a5ad68efeaba9ba0fba60ca16ea09be"
> +
> +export prefix = "/opt/ltp"
> +export exec_prefix = "/opt/ltp"
> +
> +EXTRA_OEMAKE:append = " \
> +    KERNEL_USR_INC=${WORKDIR} \
> +    ALSA_INCPATH=${STAGING_INCDIR} \
> +    ALSA_LIBPATH=${STAGING_LIBDIR} \
> +"
> +
> +RDEPENDS:${PN} += "\
> +    acl \
> +    at \
> +    pm-qa \
> +    serialcheck \
> +    memtester \
> +"
> +
> +do_install:prepend() {
> +	# Upstream ltp recipe wants to remove this test case in do_install
> +	install -d ${D}${prefix}/runtest/
> +	echo "memcg_stress" >> ${D}${prefix}/runtest/controllers
> +}
> -- 
> 2.17.1

On 4/22/2023 17:18, Denys Dmytriyenko wrote:
> On Sat, Apr 22, 2023 at 05:07:22PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>> The internal testing team would like to move to this version as the
>> basis for our internal ltp-ddt.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>> v2: Add missing space on :append line.
>>
>>   .../ltp-ddt/ltp-ddt_20230127.bb               | 47 +++++++++++++++++++
>>   1 file changed, 47 insertions(+)
>>   create mode 100644 meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
>>
>> diff --git a/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb b/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
>> new file mode 100644
>> index 00000000..10b93a87
>> --- /dev/null
>> +++ b/meta-arago-extras/recipes-devtools/ltp-ddt/ltp-ddt_20230127.bb
>> @@ -0,0 +1,47 @@
>> +FILESEXTRAPATHS:prepend := "${COREBASE}/meta/recipes-extended/ltp/ltp:"
>> +FILESEXTRAPATHS:prepend := "${ARAGOBASE}/meta-arago-extras/recipes-extended/ltp/ltp:"
>> +
>> +require recipes-extended/ltp/ltp_${PV}.bb
>> +
>> +SUMMARY = "Embedded Linux Device Driver Tests based on Linux Test Project"
>> +HOMEPAGE = "https://git.ti.com/cgit/test-automation/ltp-ddt/"
>> +
>> +PROVIDES += "ltp"
>> +
>> +DEPENDS += "alsa-lib"
>> +
>> +PE = "1"
>> +PR = "r7"
>> +PV:append = "+git${SRCPV}"
>> +
>> +SRCREV = "7f2fc4b7c972f2f188836c739a7175fad2f2b288"
>> +BRANCH ?= "master"
>> +
>> +SRC_URI:remove = "git://github.com/linux-test-project/ltp.git;branch=master;protocol=https"
>> +SRC_URI:prepend = "git://git.ti.com/git/test-automation/ltp-ddt.git;protocol=https;branch=${BRANCH} "
>> +
>> +SRC_URI:append = "https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/plain/include/uapi/linux/rpmsg_rpc.h?h=ti-linux-5.10.y;name=rpmsg_rpc;subdir=linux;downloadfilename=rpmsg_rpc.h "
> 
> Gulp. While both :prepend and :append need to ensure whitespaces are
> explicitly added to separate entries in the list, those are needed at
> different places. It is tailing space for :prepend and leading space
> for :append, of course. Or use += here to be safe...

Yeah...  It's been a long week and I'm rushing.  I was only using append 
because two lines above used prepend and I was trying to be consistent. 
v3 incoming.

> 
>> +SRC_URI[rpmsg_rpc.sha256sum] = "cd237f40a37520a1f2df19fbfeefd00c0a5ad68efeaba9ba0fba60ca16ea09be"
>> +
>> +export prefix = "/opt/ltp"
>> +export exec_prefix = "/opt/ltp"
>> +
>> +EXTRA_OEMAKE:append = " \
>> +    KERNEL_USR_INC=${WORKDIR} \
>> +    ALSA_INCPATH=${STAGING_INCDIR} \
>> +    ALSA_LIBPATH=${STAGING_LIBDIR} \
>> +"
>> +
>> +RDEPENDS:${PN} += "\
>> +    acl \
>> +    at \
>> +    pm-qa \
>> +    serialcheck \
>> +    memtester \
>> +"
>> +
>> +do_install:prepend() {
>> +	# Upstream ltp recipe wants to remove this test case in do_install
>> +	install -d ${D}${prefix}/runtest/
>> +	echo "memcg_stress" >> ${D}${prefix}/runtest/controllers
>> +}
>> -- 
>> 2.17.1