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 --]
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox