Re: [Openvpn-devel] [tap-windows 05/11] build: set default to newer ddk

[Samuli Seppänen]

I think we can drop WinDDK <= 5600 support entirely. If I'm not
mistaken, this means supporting Visual Studio 2008 and later only, which
is fine. Afaik the only "reason" to support Visual Studio 2005 and older
is Win2k support, which we don't need. OpenVPN 2.1.3 was the last
version with Win2k support anyways.

Besides that, it's an ACK.

-- 
Samuli Seppänen
Community Manager
OpenVPN Technologies, Inc

irc freenode net: mattock


> Signed-off-by: Alon Bar-Lev 
> ---
>  src/prototypes.h |2 +-
>  src/tapdrvr.c|5 +
>  2 files changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/src/prototypes.h b/src/prototypes.h
> index 55454d5..18c566f 100755
> --- a/src/prototypes.h
> +++ b/src/prototypes.h
> @@ -202,7 +202,7 @@ VOID HookDispatchFunctions();
>  
>  #if ENABLE_NONADMIN
>  
> -#if DDKVER_MAJOR < 5600
> +#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600
>  /*
>   * Better solution for use on Vista DDK, but possibly not compatible with
>   * earlier DDKs:
> diff --git a/src/tapdrvr.c b/src/tapdrvr.c
> index 0df076a..0f75b77 100755
> --- a/src/tapdrvr.c
> +++ b/src/tapdrvr.c
> @@ -36,9 +36,6 @@
>  //==
>  
>  #include "common.h"
> -#ifndef DDKVER_MAJOR
> -#error DDKVER_MAJOR must be defined as the major number of the DDK Version
> -#endif
>  
>  #define NDIS_MINIPORT_DRIVER
>  #define BINARY_COMPATIBLE 0
> @@ -66,7 +63,7 @@
>  //
>  #define ENABLE_NONADMIN 1
>  
> -#if DDKVER_MAJOR < 5600
> +#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600
>  #include 
>  #include 
>  #include 

[Openvpn-devel] [tap-windows 05/11] build: set default to newer ddk

[Alon Bar-Lev]

Signed-off-by: Alon Bar-Lev 
---
 src/prototypes.h |2 +-
 src/tapdrvr.c|5 +
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/prototypes.h b/src/prototypes.h
index 55454d5..18c566f 100755
--- a/src/prototypes.h
+++ b/src/prototypes.h
@@ -202,7 +202,7 @@ VOID HookDispatchFunctions();

 #if ENABLE_NONADMIN

-#if DDKVER_MAJOR < 5600
+#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600
 /*
  * Better solution for use on Vista DDK, but possibly not compatible with
  * earlier DDKs:
diff --git a/src/tapdrvr.c b/src/tapdrvr.c
index 0df076a..0f75b77 100755
--- a/src/tapdrvr.c
+++ b/src/tapdrvr.c
@@ -36,9 +36,6 @@
 //==

 #include "common.h"
-#ifndef DDKVER_MAJOR
-#error DDKVER_MAJOR must be defined as the major number of the DDK Version
-#endif

 #define NDIS_MINIPORT_DRIVER
 #define BINARY_COMPATIBLE 0
@@ -66,7 +63,7 @@
 //
 #define ENABLE_NONADMIN 1

-#if DDKVER_MAJOR < 5600
+#if defined(DDKVER_MAJOR) && DDKVER_MAJOR < 5600
 #include 
 #include 
 #include 
-- 
1.7.3.4