diff mbox series

[meta-arago,scarthgap] openssl-perf: Add openssl_aes256_encdec.sh script

Message ID 20240701143728.26618-1-reatmon@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,scarthgap] openssl-perf: Add openssl_aes256_encdec.sh script | expand

Commit Message

Ryan Eatmon July 1, 2024, 2:37 p.m. UTC
This is an additional openssl script that our testing flow requires.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../recipes-benchmark/openssl-perf/openssl-perf.bb        | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Denys Dmytriyenko July 1, 2024, 9:21 p.m. UTC | #1
On Mon, Jul 01, 2024 at 09:37:28AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> This is an additional openssl script that our testing flow requires.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../recipes-benchmark/openssl-perf/openssl-perf.bb        | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
> index 02f41f41..d1c51528 100644
> --- a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
> +++ b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
> @@ -1,5 +1,5 @@
> -SUMMARY = "Benchmark for checking various OpenSSL performance functions"
> -HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh"
> +SUMMARY = "Benchmarks for checking various OpenSSL performance functions"
> +HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/"
>  LICENSE = "CC-BY-SA-3.0"
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>  
> @@ -7,9 +7,11 @@ PV = "1.0"
>  
>  SRC_URI = " \
>      https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh;name=openssl_perf \
> +    https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_aes256_encdec_scripts/openssl_aes256_encdec.sh;name=openssl_aes256_encdec \
>      https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \
>  "
>  SRC_URI[openssl_perf.sha256sum] = "1ba5188beb6a1fa69d9f34df12c596ed49907c2b2d2ce2d985a305b9ecbed800"
> +SRC_URI[openssl_aes256_encdec.sha256sum] = "c63efc35f7c161e02ef6d53b428bfbfce7ea3f2d329c552dbb04f865317eb408"
>  SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3"
>  
>  S = "${WORKDIR}"
> @@ -17,7 +19,9 @@ S = "${WORKDIR}"
>  do_install() {
>      install -d ${D}${bindir}
>      install -m 0755 ${S}/openssl_perf.sh ${D}${bindir}/openssl_perf.sh
> +    install -m 0755 ${S}/openssl_aes256_encdec.sh ${D}${bindir}/openssl_aes256_encdec.sh
>  }
>  
>  FILES:${PN} = "${bindir}/openssl_perf.sh"
> +FILES:${PN} = "${bindir}/openssl_aes256_encdec.sh"

Should be += here to include both files:

ERROR: openssl-perf-1.0-r0 do_package: QA Issue: openssl-perf: Files/directories were installed but not shipped in any package:
  /usr/bin/openssl_perf.sh
Ryan Eatmon July 1, 2024, 9:28 p.m. UTC | #2
On 7/1/2024 4:21 PM, Denys Dmytriyenko wrote:
> On Mon, Jul 01, 2024 at 09:37:28AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>> This is an additional openssl script that our testing flow requires.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   .../recipes-benchmark/openssl-perf/openssl-perf.bb        | 8 ++++++--
>>   1 file changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
>> index 02f41f41..d1c51528 100644
>> --- a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
>> +++ b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
>> @@ -1,5 +1,5 @@
>> -SUMMARY = "Benchmark for checking various OpenSSL performance functions"
>> -HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh"
>> +SUMMARY = "Benchmarks for checking various OpenSSL performance functions"
>> +HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/"
>>   LICENSE = "CC-BY-SA-3.0"
>>   LIC_FILES_CHKSUM = "file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
>>   
>> @@ -7,9 +7,11 @@ PV = "1.0"
>>   
>>   SRC_URI = " \
>>       https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh;name=openssl_perf \
>> +    https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_aes256_encdec_scripts/openssl_aes256_encdec.sh;name=openssl_aes256_encdec \
>>       https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \
>>   "
>>   SRC_URI[openssl_perf.sha256sum] = "1ba5188beb6a1fa69d9f34df12c596ed49907c2b2d2ce2d985a305b9ecbed800"
>> +SRC_URI[openssl_aes256_encdec.sha256sum] = "c63efc35f7c161e02ef6d53b428bfbfce7ea3f2d329c552dbb04f865317eb408"
>>   SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3"
>>   
>>   S = "${WORKDIR}"
>> @@ -17,7 +19,9 @@ S = "${WORKDIR}"
>>   do_install() {
>>       install -d ${D}${bindir}
>>       install -m 0755 ${S}/openssl_perf.sh ${D}${bindir}/openssl_perf.sh
>> +    install -m 0755 ${S}/openssl_aes256_encdec.sh ${D}${bindir}/openssl_aes256_encdec.sh
>>   }
>>   
>>   FILES:${PN} = "${bindir}/openssl_perf.sh"
>> +FILES:${PN} = "${bindir}/openssl_aes256_encdec.sh"
> 
> Should be += here to include both files:

Good catch.  I'll fix both patches.


> ERROR: openssl-perf-1.0-r0 do_package: QA Issue: openssl-perf: Files/directories were installed but not shipped in any package:
>    /usr/bin/openssl_perf.sh
>
diff mbox series

Patch

diff --git a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
index 02f41f41..d1c51528 100644
--- a/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
+++ b/meta-arago-test/recipes-benchmark/openssl-perf/openssl-perf.bb
@@ -1,5 +1,5 @@ 
-SUMMARY = "Benchmark for checking various OpenSSL performance functions"
-HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh"
+SUMMARY = "Benchmarks for checking various OpenSSL performance functions"
+HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/cryptos_apps_scripts/"
 LICENSE = "CC-BY-SA-3.0"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9"
 
@@ -7,9 +7,11 @@  PV = "1.0"
 
 SRC_URI = " \
     https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_perf_scripts/openssl_perf.sh;name=openssl_perf \
+    https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/cryptos_apps_scripts/openssl_aes256_encdec_scripts/openssl_aes256_encdec.sh;name=openssl_aes256_encdec \
     https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \
 "
 SRC_URI[openssl_perf.sha256sum] = "1ba5188beb6a1fa69d9f34df12c596ed49907c2b2d2ce2d985a305b9ecbed800"
+SRC_URI[openssl_aes256_encdec.sha256sum] = "c63efc35f7c161e02ef6d53b428bfbfce7ea3f2d329c552dbb04f865317eb408"
 SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3"
 
 S = "${WORKDIR}"
@@ -17,7 +19,9 @@  S = "${WORKDIR}"
 do_install() {
     install -d ${D}${bindir}
     install -m 0755 ${S}/openssl_perf.sh ${D}${bindir}/openssl_perf.sh
+    install -m 0755 ${S}/openssl_aes256_encdec.sh ${D}${bindir}/openssl_aes256_encdec.sh
 }
 
 FILES:${PN} = "${bindir}/openssl_perf.sh"
+FILES:${PN} = "${bindir}/openssl_aes256_encdec.sh"