On Fri, Aug 21, 2015 at 12:59 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Mon, Aug 17, 2015 at 03:09:46PM -0700, Alex Wang wrote:
>> The 'ovs-vtep' simulator is not ported to Windows.  So, for now,
>> just skip all tests in ovn-controller-vtep.at when running in
>> Windows.
>>
>> Signed-off-by: Alex Wang <al...@nicira.com>
>
> ...
>
>> -  [OVS_RUNDIR=`pwd`; export OVS_RUNDIR
>> +  [
>> +   # this will cause skip when 'make check' using Windows setup.
>> +   AT_SKIP_IF([test $HAVE_PYTHON = no])
>
> Is it a Python or a Windows incompatibility?  If it's really the latter,
> then I'd recommend using "$IS_WIN32" = yes" instead.

My thinking was along the lines that all tests that use python code
need to have HAVE_PYTHON test. That has been the strategy for all
other python tests being skipped in Windows to prevent many individual
tests from having IS_WIN32 check in addition to HAVE_PYTHON test. I am
fine either way.

>
> Acked-by: Ben Pfaff <b...@nicira.com>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to