From d2c69afb68dac185046e781de3ba87dad85aee4c Mon Sep 17 00:00:00 2001
From: Leon Chen <32035063+LeonCHEN233@users.noreply.github.com>
Date: Wed, 28 Feb 2018 00:05:47 +0000
Subject: [PATCH] Update CMakeLists.txt

---
 CMakeLists.txt | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 50830fa618..69b2387ab9 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -12,6 +12,7 @@ option(MATLAB_BINDINGS "Compile MATLAB bindings if MATLAB is found." OFF)
 option(TEST_VERBOSE "Run test cases with verbose output." OFF)
 option(BUILD_TESTS "Build tests." ON)
 option(BUILD_CLI_EXECUTABLES "Build command-line executables." ON)
+option(BUILD_PYTHON_BINDINGS "Build Python bindings." ON)
 option(BUILD_SHARED_LIBS
     "Compile shared libraries (if OFF, static libraries are compiled)." ON)
 option(BUILD_WITH_COVERAGE
@@ -23,13 +24,11 @@ option(FORCE_CXX11
     OFF)
 enable_testing()
 
-# Python can not be binged with mlpack-master on windows yet
+# Currently Python bindings aren't supported on windows
 # Set BUILD_PYTHON_BINDINGS to OFF when the platform are windows
 if(WIN32)
   option(BUILD_PYTHON_BINDINGS "Build Python bindings." OFF)
   message(WARNING "By default Python bindings are not compiled for Windows because they are not known to work.  Set BUILD_PYTHON_BINDINGS to ON if you want them built.")
-else()
-  option(BUILD_PYTHON_BINDINGS "Build Python bindings." ON)
 endif()
 
 # Ensure that we have a C++11 compiler.  In newer versions of CMake, this is