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.li...@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):
> 

Attachment: signature.asc
Description: OpenPGP digital signature

-- 
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to