Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mikko Rapeli <mikko.rapeli@linaro.org>
To: barne.carstensen@danfoss.com,
	openembedded-core@lists.openembedded.org,
	randy.macleod@windriver.com
Subject: Re: [OE-core] [PATCH] Fixes [YOCTO #15965]
Date: Tue, 16 Sep 2025 16:41:14 +0300	[thread overview]
Message-ID: <aMlo-hknGOEJqrGP@nuoska> (raw)
In-Reply-To: <1865C555FCF7D165.11775@lists.openembedded.org>

Hi,

On Tue, Sep 16, 2025 at 04:09:58PM +0300, Mikko Rapeli via lists.openembedded.org wrote:
> This is not for master branch and subject line should be
> something like "[branch name][PATCH] testexport.bbclass: always capture ignore files"
> 
> 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)
> > 
> > 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)
> 
> Master branch is quite different. There the tests and their data
> files are now copied from all enabled layers, not just core.
> So I would rather port those fixes to relevant branches to avoid
> duplicating maitenance.

Since the commits are bit scattered, these from master branch
would be needed if not already in the relevant branch:

e5e8f2aa0a63d testexport.bbclass oe-test: capture all tests and data from all layers
05e97d5e6e24b oeqa context.py: use TEST_SUITES if set
2d900c3061e12 oeqa/runtime: set self.runner and handle None

Cheers,

-Mikko


  parent reply	other threads:[~2025-09-16 13:41 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 [this message]
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
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=aMlo-hknGOEJqrGP@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