[yocto] [PATCH v2][yocto-autobuilder] RunOeSelftest.py: changed to new oe-selftest syntax
Aníbal Limón
anibal.limon at linux.intel.com
Thu Sep 1 12:24:49 PDT 2016
Hi Humberto,
I acked the patch only a few comments, you forget to sign your patch,
please do with git commit -s.
Only for be clear this patch will support the next oe-selftest cmdline
also the old one so it could be integrated without the integration of
the oe-selttest new cmdline patch.
Cheers,
alimon
On 09/01/2016 01:56 PM, Humberto Ibarra wrote:
> oe-selftest is changing syntax with YOCTO #8938. This patch checks the correct syntax version and uses the right command to avoid breaking oe-seltest autobuilder. Should be integrated together with the syntax changing patch.
Acked-by: Aníbal Limón <anibal.limon at linux.intel.com>
>
> [YOCTO #8938]
> ---
> .../autobuilder/buildsteps/RunOeSelftest.py | 16 ++++++++++++++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/lib/python2.7/site-packages/autobuilder/buildsteps/RunOeSelftest.py b/lib/python2.7/site-packages/autobuilder/buildsteps/RunOeSelftest.py
> index 809bcc3..ce0c0f6 100644
> --- a/lib/python2.7/site-packages/autobuilder/buildsteps/RunOeSelftest.py
> +++ b/lib/python2.7/site-packages/autobuilder/buildsteps/RunOeSelftest.py
> @@ -34,9 +34,21 @@ class RunOeSelftest(ShellCommand):
> self.command += "netstat -an | grep -q 5901 ;"
> self.command += "if [ $? -ne 0 ]; then echo 'Starting a VNC server on :1'; vncserver :1; else echo 'Will use a VNC server already running on :1'; fi;"
> if self.tests == None:
> - self.command += "if [ -d ../meta-selftest ]; then export DISPLAY=:1; oe-selftest --run-all-tests; else echo 'Skipping step - no meta-selftest layer here'; fi"
> + oe_selftest_cmd = ('run', '--run-all-tests')
> else:
> - self.command += "if [ -d ../meta-selftest ]; then export DISPLAY=:1; oe-selftest " + self.tests + "; else echo 'Skipping step - no meta-selftest layer here'; fi"
> + oe_selftest_cmd = ("run --tests %s" % self.tests, "--run-tests %s" % self.tests)
> + self.command += """ \
> +if [ -d ../meta-selftest ]; then \
> + export DISPLAY=:1; \
> + VERSION=$(oe-selftest --version 2> /dev/null | cut -d' ' -f2); \
> + if [ \"$VERSION\" == '2.0' ]; then \
> + oe-selftest %s; \
> + else \
> + oe-selftest %s; \
> + fi; \
> +else \
> + echo 'Skipping step - no meta-selftest layer here';
> +fi""" % oe_selftest_cmd
> ShellCommand.start(self)
>
> def _createOESelftestErrorReport(self, log):
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.yoctoproject.org/pipermail/yocto/attachments/20160901/4e1cc69a/attachment.pgp>
More information about the yocto
mailing list