ltp: remove open-posix-testsuite build logs

Message ID 20220707164313.2697033-1-ross.burton@arm.com
State Accepted, archived
Commit dc5141b86cfcad0daa56d4b0cac9fd57e83b515a
Headers show
Series ltp: remove open-posix-testsuite build logs | expand

Commit Message

Ross Burton July 7, 2022, 4:43 p.m. UTC
We don't need to package the open-posix-testsuite build logs. Typically
these are just lines of "SUCCESS" but sometimes there will be compile
warnings, which will include build paths.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/recipes-extended/ltp/ltp_20220527.bb | 3 +++
 1 file changed, 3 insertions(+)

Comments

Khem Raj July 7, 2022, 5:07 p.m. UTC | #1
On Thu, Jul 7, 2022 at 12:43 PM Ross Burton <ross.burton@arm.com> wrote:
>
> We don't need to package the open-posix-testsuite build logs. Typically
> these are just lines of "SUCCESS" but sometimes there will be compile
> warnings, which will include build paths.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
>  meta/recipes-extended/ltp/ltp_20220527.bb | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/meta/recipes-extended/ltp/ltp_20220527.bb b/meta/recipes-extended/ltp/ltp_20220527.bb
> index b3ef8f59105..e3abfd01e1c 100644
> --- a/meta/recipes-extended/ltp/ltp_20220527.bb
> +++ b/meta/recipes-extended/ltp/ltp_20220527.bb
> @@ -79,6 +79,9 @@ do_install(){
>      # The controllers memcg_stree test seems to cause us hangs and takes 900s
>      # (maybe we expect more regular output?), anyhow, skip it
>      sed -e '/^memcg_stress/d' -i ${D}${prefix}/runtest/controllers
> +
> +    # We don't need to ship the compile logs that open_posix_testsuite writes
> +    rm -f ${D}${prefix}/testcases/open_posix_testsuite/logfile.*
>  }
>

i was seeing build QA warnings this should help. Is this the only file
though I wonder since i saw a bunch of messages
>  RDEPENDS:${PN} = "\
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#167792): https://lists.openembedded.org/g/openembedded-core/message/167792
> Mute This Topic: https://lists.openembedded.org/mt/92232580/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Ross Burton July 8, 2022, 11:05 a.m. UTC | #2
On 7 Jul 2022, at 18:07, Khem Raj <raj.khem@gmail.com> wrote:
> i was seeing build QA warnings this should help. Is this the only file
> though I wonder since i saw a bunch of messages

This is the only one that the autobuilder saw at least.

Ross

Patch

diff --git a/meta/recipes-extended/ltp/ltp_20220527.bb b/meta/recipes-extended/ltp/ltp_20220527.bb
index b3ef8f59105..e3abfd01e1c 100644
--- a/meta/recipes-extended/ltp/ltp_20220527.bb
+++ b/meta/recipes-extended/ltp/ltp_20220527.bb
@@ -79,6 +79,9 @@  do_install(){
     # The controllers memcg_stree test seems to cause us hangs and takes 900s
     # (maybe we expect more regular output?), anyhow, skip it
     sed -e '/^memcg_stress/d' -i ${D}${prefix}/runtest/controllers
+
+    # We don't need to ship the compile logs that open_posix_testsuite writes
+    rm -f ${D}${prefix}/testcases/open_posix_testsuite/logfile.*
 }
 
 RDEPENDS:${PN} = "\