Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mikko Rapeli <mikko.rapeli@linaro.org>
To: barne.carstensen@danfoss.com
Cc: openembedded-core@lists.openembedded.org, randy.macleod@windriver.com
Subject: Re: [OE-core] [PATCH] Fixes [YOCTO #15965]
Date: Wed, 17 Sep 2025 12:46:54 +0300	[thread overview]
Message-ID: <aMqDjoK4oj_m0-04@nuoska> (raw)
In-Reply-To: <20250916122517.1965-1-barne.carstensen@danfoss.com>

Hi,

On Tue, Sep 16, 2025 at 12:25:22PM +0000, Barne Carstensen via lists.openembedded.org wrote:
> Copy all the parselogs-ignores file so that they are always packed into the testexport.tar.gz (even when testsuite parselog is not selected)

This hints to the old behavior of testexport which was taking into account
TEST_SUITES variable. This is no longer the case in poky master branch.
All tests and related files like parselogs ignore lists are copied
as is to testexport tar ball, and from all enabled layers.

> Signed-off-by: Barne Carstensen <barne.carstensen@danfoss.com>
> CC: randy.macleod@windriver.com
> ---
>  meta/classes-recipe/testexport.bbclass | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/meta/classes-recipe/testexport.bbclass b/meta/classes-recipe/testexport.bbclass
> index 843d777e3b..2f473a8811 100644
> --- a/meta/classes-recipe/testexport.bbclass
> +++ b/meta/classes-recipe/testexport.bbclass
> @@ -79,6 +79,7 @@ def testexport_main(d):
>  def copy_needed_files(d, tc):
>      import shutil
>      import oe.path
> +    import glob
>  
>      from oeqa.utils.package_manager import _get_json_file
>      from oeqa.core.utils.test import getSuiteCasesFiles
> @@ -120,6 +121,11 @@ def copy_needed_files(d, tc):
>                      oe.path.copytree(src, dst)
>                  else:
>                      shutil.copy2(src, dst)
> +    
> +    # Copy all parselogs-ignores files
> +    parselogs_ignores = glob.glob(os.path.join(corebase_path, 'meta', 'lib', 'oeqa', 'runtime', 'cases', 'parselogs-ignores-*.txt'))
> +    for f in parselogs_ignores:
> +         shutil.copy2(f, cases_path)

This does not seem right. The lines just above should copy
the path "lib/oeqa/runtime" from all enabled layers with
oe.path.copytree(). Thus it should not be necessary to copy
parselogs files separately using a glob.

Can you try again with master branch without your patch?

If the issue still is there and parselogs files are missing
from testexport tar ball, is the layer in question not enabled
for building in BBLAYERS?

Cheers,

-Mikko


  parent reply	other threads:[~2025-09-17  9:47 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-16 12:25 [PATCH] Fixes [YOCTO #15965] Barne Carstensen
2025-09-16 12:30 ` Patchtest results for " patchtest
2025-09-16 13:09 ` [OE-core] " Mikko Rapeli
     [not found] ` <1865C555FCF7D165.11775@lists.openembedded.org>
2025-09-16 13:41   ` Mikko Rapeli
2025-09-17  7:26     ` Barne Carstensen
2025-09-17  8:33       ` Quentin Schulz
     [not found]         ` <DB9PR01MB72102C7EF8F8862F583523158717A@DB9PR01MB7210.eurprd01.prod.exchangelabs.com>
2025-09-17  9:24           ` Quentin Schulz
2025-09-18  8:18             ` Barne Carstensen
2025-09-17  9:21       ` Mikko Rapeli
2025-09-17  9:46 ` Mikko Rapeli [this message]
2025-09-18  8:05   ` Barne Carstensen
2025-09-18  8:09     ` Mikko Rapeli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aMqDjoK4oj_m0-04@nuoska \
    --to=mikko.rapeli@linaro.org \
    --cc=barne.carstensen@danfoss.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=randy.macleod@windriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox