All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joshua Lock <joshua.g.lock@linux.intel.com>
To: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>,
	 "Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 4/9] selftest: skip some devtool tests when distro is poky-tiny due to missing libx11
Date: Tue, 18 Oct 2016 17:39:35 +0100	[thread overview]
Message-ID: <1476808775.14120.7.camel@linux.intel.com> (raw)
In-Reply-To: <ad1b1d7c-9789-7560-283f-bdd6f36e5362@linux.intel.com>

[-- Attachment #1: Type: text/plain, Size: 1845 bytes --]

On Mon, 2016-10-17 at 10:19 -0500, Leonardo Sandoval wrote:
> 
>     
> 
>     
>     
> 
>     On 10/17/2016 06:42 AM, Burton, Ross
>       wrote:
> 
>     
> 
> >       
> >         
> > 
> > > >           On 14 October 2016 at 17:40, <leonardo.sandoval.gonzalez@
linux.intel.com>
> >             wrote:
> > 
> > >                  
> > >                  def test_devtool_add_fetch_git(self):
> > > 
> > >                 -        # Fetch source
> > > 
> > >                 +        if self.distro == 'poky-tiny':
> > > 
> > >                 +            self.skipTest('libmatchbox2 is not
> > > > > >                 buildable with poky-tiny because it requires a
libx11
> > >                 provider')
> > > 
> > > > > >                          tempdir =
tempfile.mkdtemp(prefix='devtoolqa')
> > > 
> > >                          self.track_for_cleanup(tempdir)
> > > 
> > > > > >                          url =
'git://git.yoctoproject.org/libmatchbox'
> > > 
> > >             
> >           
> > 
> >           
> > 
> >           Changing the test to use a recipe that doesn't require a
> > > >           distro feature that many distros disable seems like a
more
> >           sustainable fix.
> > 
> >         
> > 
> >         
> > 
> >       
> > 
>     Which recipe do you think would be more generic?

Something in the minimal image which doesn't rely on DISTRO_FEATURES
which aren't in the default distro.

Thanks,

Joshua

[-- Attachment #2: Type: text/html, Size: 2327 bytes --]

  reply	other threads:[~2016-10-18 16:39 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-14 16:40 [PATCH 0/9] Support supported distros on selftest leonardo.sandoval.gonzalez
2016-10-14 16:40 ` [PATCH 1/9] selftest: set correct linux-yocto recipe when distro is poky-tiny leonardo.sandoval.gonzalez
2016-10-17 11:30   ` Burton, Ross
2016-10-17 14:34     ` Leonardo Sandoval
2016-10-14 16:40 ` [PATCH 2/9] selftest: skip two bbtest tests depending on distro leonardo.sandoval.gonzalez
2016-10-17  7:08   ` Joshua Lock
2016-10-17 16:04     ` Leonardo Sandoval
2016-10-18 16:38       ` Joshua Lock
2016-10-14 16:40 ` [PATCH 3/9] selftest: skip test_read_only_image on poky-tiny distro leonardo.sandoval.gonzalez
2016-10-17 11:48   ` Burton, Ross
2016-10-17 15:00     ` Leonardo Sandoval
2016-10-14 16:40 ` [PATCH 4/9] selftest: skip some devtool tests when distro is poky-tiny due to missing libx11 leonardo.sandoval.gonzalez
2016-10-17 11:42   ` Burton, Ross
2016-10-17 15:19     ` Leonardo Sandoval
2016-10-18 16:39       ` Joshua Lock [this message]
2016-10-14 16:40 ` [PATCH 5/9] selftest: skip those imagefeatures tests not relevant for certain distros leonardo.sandoval.gonzalez
2016-10-17 11:44   ` Burton, Ross
2016-10-14 16:40 ` [PATCH 6/9] selftest: skip test_recipetool_create_git on poky-tiny distro leonardo.sandoval.gonzalez
2016-10-17 11:43   ` Burton, Ross
2016-10-14 16:40 ` [PATCH 7/9] selftest: skip runtime-test's test_testimage_install " leonardo.sandoval.gonzalez
2016-10-17  7:09   ` Joshua Lock
2016-10-17 16:02     ` Leonardo Sandoval
2016-10-18 16:40       ` Joshua Lock
2016-10-14 16:40 ` [PATCH 8/9] selftest: skip sstatetests methods considering distro and its features leonardo.sandoval.gonzalez
2016-10-14 16:40 ` [PATCH 9/9] selftest: skip wic test methods for poky-tiny distro leonardo.sandoval.gonzalez
2016-11-24 20:55 ` [PATCH v2 00/13] Skip selftests depending on distro and its features leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 01/13] selftest: base: new object member to store the DISTRO value leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 02/13] selftest: bbtests: use minimal image so all distros can execute it leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 03/13] selftest: bbtests: run non-gplv3 test only on relevant distros leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 04/13] selftest: buildoptions: skip read-only-image test depending on distro leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 05/13] selftest: buildoptions: skip test in case features are missing leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 06/13] selftest: devtool: use mraa instead of libmatchbox to lessen distro requirements leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 07/13] selftest: imagefeatures: skip tests in case distro feature is missing leonardo.sandoval.gonzalez
2016-12-13 15:05     ` Burton, Ross
2016-11-24 20:58   ` [PATCH v2 08/13] selftest: imagefeatures: skip bmap test in case of poky-tiny leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 09/13] selftest: recipetool: use mraa instead of libmatchbox to lessen distro requirements leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 10/13] selftest: runtime-test: skip image-install test for poky-tiny leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 11/13] selftest: sstatetests: skip methods in case of poky-tiny or opengl is missing leonardo.sandoval.gonzalez
2016-12-13 15:25     ` Burton, Ross
2016-12-15 20:21       ` Leonardo Sandoval
2016-11-24 20:58   ` [PATCH v2 12/13] selftest: wic: skip tests for poky-tiny distro policy leonardo.sandoval.gonzalez
2016-11-24 20:58   ` [PATCH v2 13/13] selftest: bblayers: remove linux kernel checks for show-recipes check leonardo.sandoval.gonzalez

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=1476808775.14120.7.camel@linux.intel.com \
    --to=joshua.g.lock@linux.intel.com \
    --cc=leonardo.sandoval.gonzalez@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.