From: "Robert P. J. Day" <rpjday@crashcourse.ca>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: Yocto discussion list <yocto@yoctoproject.org>
Subject: Re: i assume "ptest" is an all-or-nothing selection, yes?
Date: Tue, 5 Aug 2014 15:37:27 -0400 (EDT) [thread overview]
Message-ID: <alpine.LFD.2.11.1408051449500.22646@localhost> (raw)
In-Reply-To: <1407232532.6981.76.camel@ted>
On Tue, 5 Aug 2014, Richard Purdie wrote:
> On Tue, 2014-08-05 at 05:40 -0400, Robert P. J. Day wrote:
> > dev manual, section 5.13.6, clearly implies that ptest is selected
> > either across your build or not at all -- i see no suggestion that you
> > can enable it with, say, package overrides, correct?
>
> Correct.
i thought as much, just wanted to make sure. and while that might be
obvious to many people, i think a short note at the beginning of that
section wouldn't hurt, but a couple more questions about ptest so i
can make some adjustments all at once, based on what's written here:
http://www.yoctoproject.org/docs/current/dev-manual/dev-manual.html#testing-packages-with-ptest
toward the end of that section, there is the claim:
"However, you still must add a do_compile_ptest function to build the
test suite. Add a function similar to the following to your recipe:
do_compile_ptest() {
oe_runmake buildtest-TESTS
}
well, not really ... i did some perusing and there are numerous
ptest-enabled recipes that don't define their own do_compile_ptest()
routine, obviously because the standard compile step handles the
compilation of the test code.
and, finally, there's this last claim:
"The ptest class automatically copies the file run-ptest to the target
and then runs make install-ptest to run the tests. If this is not
enough, you need to create a do_install_ptest function and make sure
it gets called after the "make install-ptest" completes."
couple things there. rather than define a do_install_ptest()
routine, i notice a number of recipes simply patch their Makefile to
add the appropriate install-ptest: target, so if one is clever enough
to write the proper Makefile target, one could have a ptest-enabled
recipe with no definition whatsoever of *any* of the do_*_ptest()
routines.
oh, and about that last claim:
"If this is not enough, you need to create a do_install_ptest function
and make sure it gets called after the "make install-ptest"
completes."
don't know what you mean by "make sure it gets called ..." since the
code in ptest.bbclass:
if grep -q install-ptest: Makefile; then
oe_runmake DESTDIR=${D}${PTEST_PATH} install-ptest
fi
do_install_ptest
pretty much guarantees that it will be called -- there is no need for
the developer to "make sure" of it, yes?
anyway, i think i have a handle on this. any thoughts on the above?
rday
--
========================================================================
Robert P. J. Day Ottawa, Ontario, CANADA
http://crashcourse.ca
Twitter: http://twitter.com/rpjday
LinkedIn: http://ca.linkedin.com/in/rpjday
========================================================================
prev parent reply other threads:[~2014-08-05 19:37 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-05 9:40 i assume "ptest" is an all-or-nothing selection, yes? Robert P. J. Day
2014-08-05 9:55 ` Richard Purdie
2014-08-05 19:37 ` Robert P. J. Day [this message]
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=alpine.LFD.2.11.1408051449500.22646@localhost \
--to=rpjday@crashcourse.ca \
--cc=richard.purdie@linuxfoundation.org \
--cc=yocto@yoctoproject.org \
/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.