diff mbox

[ovs-dev,V2,5/5] Python tests: Enable python tests on Windows

Message ID 1483456542-7522-6-git-send-email-abalutoiu@cloudbasesolutions.com
State Superseded
Headers show

Commit Message

Alin Balutoiu Jan. 3, 2017, 3:16 p.m. UTC
From: Alin Balutoiu <abalutoiu@cloudbasesolutions.com>

Updated the Readme with additional library required for
the python tests.

Remove the code which disable python tests to
be run on Windows.

Signed-off-by: Alin-Gheorghe Balutoiu <abalutoiu@cloudbasesolutions.com>
---
V2: Changed code-block from bash to console.
---
 Documentation/intro/install/windows.rst | 7 +++++++
 tests/atlocal.in                        | 8 --------
 2 files changed, 7 insertions(+), 8 deletions(-)

Comments

Alin Balutoiu Jan. 3, 2017, 4:27 p.m. UTC | #1
Please ignore this patch, I will send another one soon.

Thanks,
Alin Balutoiu.

> -----Original Message-----
> From: Alin Balutoiu
> Sent: Tuesday, January 3, 2017 5:17 PM
> To: dev@openvswitch.org
> Cc: Alin Balutoiu <abalutoiu@cloudbasesolutions.com>
> Subject: [PATCH V2 5/5] Python tests: Enable python tests on Windows
> 
> From: Alin Balutoiu <abalutoiu@cloudbasesolutions.com>
> 
> Updated the Readme with additional library required for the python tests.
> 
> Remove the code which disable python tests to be run on Windows.
> 
> Signed-off-by: Alin-Gheorghe Balutoiu <abalutoiu@cloudbasesolutions.com>
> ---
> V2: Changed code-block from bash to console.
> ---
>  Documentation/intro/install/windows.rst | 7 +++++++
>  tests/atlocal.in                        | 8 --------
>  2 files changed, 7 insertions(+), 8 deletions(-)
> 
> diff --git a/Documentation/intro/install/windows.rst
> b/Documentation/intro/install/windows.rst
> index 9bd1813..47dd68e 100644
> --- a/Documentation/intro/install/windows.rst
> +++ b/Documentation/intro/install/windows.rst
> @@ -60,6 +60,13 @@ The following explains the steps in some detail.
> 
>    Install the latest Python 2.x from python.org and verify that its path is
>    part of Windows' PATH environment variable.
> +  We require that you have Python six and pypiwin32 libraries installed.
> +  The libraries can be installed via pip command:
> +
> +   .. code-block:: console
> +
> +      $ pip install six
> +      $ pip install pypiwin32
> 
>  - Visual Studio
> 
> diff --git a/tests/atlocal.in b/tests/atlocal.in index 1353b46..9fbab39 100644
> --- a/tests/atlocal.in
> +++ b/tests/atlocal.in
> @@ -109,14 +109,6 @@ else
>      HAVE_IPV6=no
>  fi
> 
> -# XXX: Disable Python related tests on Windows because Open vSwitch
> code -# written in Python has not been ported to the Windows platform. We
> will -# need to remove the next block after porting is complete.
> -if test "$IS_WIN32" = "yes"; then
> -    HAVE_PYTHON="no"
> -    HAVE_PYTHON3="no"
> -fi
> -
>  if test "$HAVE_PYTHON" = "yes" \
>     && test "x`$PYTHON $abs_top_srcdir/tests/test-l7.py --help | grep 'ftp'`"
> != x; then
>      HAVE_PYFTPDLIB="yes"
> --
> 2.10.0.windows.1
diff mbox

Patch

diff --git a/Documentation/intro/install/windows.rst b/Documentation/intro/install/windows.rst
index 9bd1813..47dd68e 100644
--- a/Documentation/intro/install/windows.rst
+++ b/Documentation/intro/install/windows.rst
@@ -60,6 +60,13 @@  The following explains the steps in some detail.
 
   Install the latest Python 2.x from python.org and verify that its path is
   part of Windows' PATH environment variable.
+  We require that you have Python six and pypiwin32 libraries installed.
+  The libraries can be installed via pip command:
+
+   .. code-block:: console
+
+      $ pip install six
+      $ pip install pypiwin32
 
 - Visual Studio
 
diff --git a/tests/atlocal.in b/tests/atlocal.in
index 1353b46..9fbab39 100644
--- a/tests/atlocal.in
+++ b/tests/atlocal.in
@@ -109,14 +109,6 @@  else
     HAVE_IPV6=no
 fi
 
-# XXX: Disable Python related tests on Windows because Open vSwitch code
-# written in Python has not been ported to the Windows platform. We will
-# need to remove the next block after porting is complete.
-if test "$IS_WIN32" = "yes"; then
-    HAVE_PYTHON="no"
-    HAVE_PYTHON3="no"
-fi
-
 if test "$HAVE_PYTHON" = "yes" \
    && test "x`$PYTHON $abs_top_srcdir/tests/test-l7.py --help | grep 'ftp'`" != x; then
     HAVE_PYFTPDLIB="yes"