Message ID | b8478f4b34c727aca697ce40f4814b8c845d7872-HTML@gerrit.openvpn.net |
---|---|
State | Superseded |
Headers | show |
Series | [Openvpn-devel,M] Change in openvpn[master]: Remove compat versionhelpers.h and remove cmake/configure check for it | expand |
Make is original and don't change anything to make it affect please On Mon, Nov 20, 2023, 2:52 AM plaisthos (Code Review) <gerrit@openvpn.net> wrote: > Attention is currently required from: flichtenheld. > > plaisthos would like flichtenheld to *review* this change. > > View Change <http://gerrit.openvpn.net/c/openvpn/+/445?usp=email> > > Remove compat versionhelpers.h and remove cmake/configure check for it > > The cmake file defined that file to be never present in contrast to the > old msvc-config.h that always had it present. Also interactive.c includes > versionhelpers.h without the check, so we always assume it to be present > anyway. Remove also the comapt implementation taken from mingw > > Change-Id: I9c85ccab6d51064ebff2c391740ba8c2d044ed1a > --- > M CMakeLists.txt > M config.h.cmake.in > M configure.ac > M src/compat/Makefile.am > D src/compat/compat-versionhelpers.h > M src/openvpn/win32.c > M src/openvpnserv/interactive.c > 7 files changed, 1 insertion(+), 131 deletions(-) > > git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/45/445/1 > > diff --git a/CMakeLists.txt b/CMakeLists.txt > index d21c9bd..fa6d623 100644 > --- a/CMakeLists.txt > +++ b/CMakeLists.txt > @@ -308,7 +308,6 @@ > src/compat/compat-dirname.c > src/compat/compat-gettimeofday.c > src/compat/compat-strsep.c > - src/compat/compat-versionhelpers.h > src/openvpn/argv.c > src/openvpn/argv.h > src/openvpn/base64.c > diff --git a/config.h.cmake.in b/config.h.cmake.in > index 19b79bc..8edaff4 100644 > --- a/config.h.cmake.in > +++ b/config.h.cmake.in > @@ -374,9 +374,6 @@ > /* Define to 1 if you have the <valgrind/memcheck.h> header file. */ > #undef HAVE_VALGRIND_MEMCHECK_H > > -/* Define to 1 if you have the <versionhelpers.h> header file. */ > -#undef HAVE_VERSIONHELPERS_H > - > /* Define to 1 if you have the `vfork' function. */ > #undef HAVE_VFORK > > diff --git a/configure.ac b/configure.ac > index 84eaad6..94c6654 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -455,7 +455,6 @@ > unistd.h dlfcn.h \ > netinet/in.h \ > netinet/tcp.h arpa/inet.h netdb.h \ > - versionhelpers.h \ > ]) > AC_CHECK_HEADERS([ \ > sys/time.h sys/ioctl.h sys/stat.h \ > diff --git a/src/compat/Makefile.am b/src/compat/Makefile.am > index f5de451..5298dd8 100644 > --- a/src/compat/Makefile.am > +++ b/src/compat/Makefile.am > @@ -20,5 +20,4 @@ > compat-basename.c \ > compat-gettimeofday.c \ > compat-daemon.c \ > - compat-strsep.c \ > - compat-versionhelpers.h > + compat-strsep.c > \ No newline at end of file > diff --git a/src/compat/compat-versionhelpers.h b/src/compat/compat-versionhelpers.h > deleted file mode 100644 > index b071602..0000000 > --- a/src/compat/compat-versionhelpers.h > +++ /dev/null > @@ -1,116 +0,0 @@ > -/** > - * This file is part of the mingw-w64 runtime package. > - * No warranty is given; refer to the file DISCLAIMER within this package. > - */ > - > -#ifndef _INC_VERSIONHELPERS > -#define _INC_VERSIONHELPERS > - > -#include <winapifamily.h> > - > -#if WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP) && !defined(__WIDL__) > - > -#ifdef __cplusplus > -#define VERSIONHELPERAPI inline bool > -#else > -#define VERSIONHELPERAPI FORCEINLINE BOOL > -#endif > - > -#define _WIN32_WINNT_WINBLUE 0x0603 > - > -#ifndef _WIN32_WINNT_WINTHRESHOLD > -#define _WIN32_WINNT_WINTHRESHOLD 0x0A00 /* Windows 10 */ > -#endif > - > -VERSIONHELPERAPI > -IsWindowsVersionOrGreater(WORD major, WORD minor, WORD servpack) > -{ > - OSVERSIONINFOEXW vi = {sizeof(vi), major, minor, 0, 0, {0}, servpack}; > - return VerifyVersionInfoW(&vi, VER_MAJORVERSION|VER_MINORVERSION|VER_SERVICEPACKMAJOR, > - VerSetConditionMask(VerSetConditionMask(VerSetConditionMask(0, > - VER_MAJORVERSION, VER_GREATER_EQUAL), > - VER_MINORVERSION, VER_GREATER_EQUAL), > - VER_SERVICEPACKMAJOR, VER_GREATER_EQUAL)); > -} > - > -VERSIONHELPERAPI > -IsWindowsXPOrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 0); > -} > - > -VERSIONHELPERAPI > -IsWindowsXPSP1OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 1); > -} > - > -VERSIONHELPERAPI > -IsWindowsXPSP2OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 2); > -} > - > -VERSIONHELPERAPI > -IsWindowsXPSP3OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 3); > -} > - > -VERSIONHELPERAPI > -IsWindowsVistaOrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 0); > -} > - > -VERSIONHELPERAPI > -IsWindowsVistaSP1OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 1); > -} > - > -VERSIONHELPERAPI > -IsWindowsVistaSP2OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 2); > -} > - > -VERSIONHELPERAPI > -IsWindows7OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN7), LOBYTE(_WIN32_WINNT_WIN7), 0); > -} > - > -VERSIONHELPERAPI > -IsWindows7SP1OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN7), LOBYTE(_WIN32_WINNT_WIN7), 1); > -} > - > -VERSIONHELPERAPI > -IsWindows8OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN8), LOBYTE(_WIN32_WINNT_WIN8), 0); > -} > - > -VERSIONHELPERAPI > -IsWindows8Point1OrGreater(void) > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINBLUE), LOBYTE(_WIN32_WINNT_WINBLUE), 0); > -} > - > -VERSIONHELPERAPI > -IsWindows10OrGreater() > -{ > - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINTHRESHOLD), LOBYTE(_WIN32_WINNT_WINTHRESHOLD), 0); > -} > - > -VERSIONHELPERAPI > -IsWindowsServer(void) > -{ > - OSVERSIONINFOEXW vi = {sizeof(vi), 0, 0, 0, 0, {0}, 0, 0, 0, VER_NT_WORKSTATION}; > - return !VerifyVersionInfoW(&vi, VER_PRODUCT_TYPE, VerSetConditionMask(0, VER_PRODUCT_TYPE, VER_EQUAL)); > -} > - > -#endif /* if WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP) && !defined(__WIDL__) */ > -#endif /* ifndef _INC_VERSIONHELPERS */ > diff --git a/src/openvpn/win32.c b/src/openvpn/win32.c > index 47eb1fc..e998d90 100644 > --- a/src/openvpn/win32.c > +++ b/src/openvpn/win32.c > @@ -45,11 +45,7 @@ > > #include "memdbg.h" > > -#ifdef HAVE_VERSIONHELPERS_H > #include <versionhelpers.h> > -#else > -#include "compat-versionhelpers.h" > -#endif > > #include "block_dns.h" > > diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c > index a47db8a..32c8996 100644 > --- a/src/openvpnserv/interactive.c > +++ b/src/openvpnserv/interactive.c > @@ -34,11 +34,7 @@ > #include <shellapi.h> > #include <mstcpip.h> > > -#ifdef HAVE_VERSIONHELPERS_H > #include <versionhelpers.h> > -#else > -#include "compat-versionhelpers.h" > -#endif > > #include "openvpn-msg.h" > #include "validate.h" > > To view, visit change 445 > <http://gerrit.openvpn.net/c/openvpn/+/445?usp=email>. To unsubscribe, or > for help writing mail filters, visit settings > <http://gerrit.openvpn.net/settings>. > Gerrit-Project: openvpn > Gerrit-Branch: master > Gerrit-Change-Id: I9c85ccab6d51064ebff2c391740ba8c2d044ed1a > Gerrit-Change-Number: 445 > Gerrit-PatchSet: 1 > Gerrit-Owner: plaisthos <arne-openvpn@rfc2549.org> > Gerrit-Reviewer: flichtenheld <frank@lichtenheld.com> > Gerrit-CC: openvpn-devel <openvpn-devel@lists.sourceforge.net> > Gerrit-Attention: flichtenheld <frank@lichtenheld.com> > Gerrit-MessageType: newchange > _______________________________________________ > Openvpn-devel mailing list > Openvpn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/openvpn-devel >
diff --git a/CMakeLists.txt b/CMakeLists.txt index d21c9bd..fa6d623 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -308,7 +308,6 @@ src/compat/compat-dirname.c src/compat/compat-gettimeofday.c src/compat/compat-strsep.c - src/compat/compat-versionhelpers.h src/openvpn/argv.c src/openvpn/argv.h src/openvpn/base64.c diff --git a/config.h.cmake.in b/config.h.cmake.in index 19b79bc..8edaff4 100644 --- a/config.h.cmake.in +++ b/config.h.cmake.in @@ -374,9 +374,6 @@ /* Define to 1 if you have the <valgrind/memcheck.h> header file. */ #undef HAVE_VALGRIND_MEMCHECK_H -/* Define to 1 if you have the <versionhelpers.h> header file. */ -#undef HAVE_VERSIONHELPERS_H - /* Define to 1 if you have the `vfork' function. */ #undef HAVE_VFORK diff --git a/configure.ac b/configure.ac index 84eaad6..94c6654 100644 --- a/configure.ac +++ b/configure.ac @@ -455,7 +455,6 @@ unistd.h dlfcn.h \ netinet/in.h \ netinet/tcp.h arpa/inet.h netdb.h \ - versionhelpers.h \ ]) AC_CHECK_HEADERS([ \ sys/time.h sys/ioctl.h sys/stat.h \ diff --git a/src/compat/Makefile.am b/src/compat/Makefile.am index f5de451..5298dd8 100644 --- a/src/compat/Makefile.am +++ b/src/compat/Makefile.am @@ -20,5 +20,4 @@ compat-basename.c \ compat-gettimeofday.c \ compat-daemon.c \ - compat-strsep.c \ - compat-versionhelpers.h + compat-strsep.c \ No newline at end of file diff --git a/src/compat/compat-versionhelpers.h b/src/compat/compat-versionhelpers.h deleted file mode 100644 index b071602..0000000 --- a/src/compat/compat-versionhelpers.h +++ /dev/null @@ -1,116 +0,0 @@ -/** - * This file is part of the mingw-w64 runtime package. - * No warranty is given; refer to the file DISCLAIMER within this package. - */ - -#ifndef _INC_VERSIONHELPERS -#define _INC_VERSIONHELPERS - -#include <winapifamily.h> - -#if WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP) && !defined(__WIDL__) - -#ifdef __cplusplus -#define VERSIONHELPERAPI inline bool -#else -#define VERSIONHELPERAPI FORCEINLINE BOOL -#endif - -#define _WIN32_WINNT_WINBLUE 0x0603 - -#ifndef _WIN32_WINNT_WINTHRESHOLD -#define _WIN32_WINNT_WINTHRESHOLD 0x0A00 /* Windows 10 */ -#endif - -VERSIONHELPERAPI -IsWindowsVersionOrGreater(WORD major, WORD minor, WORD servpack) -{ - OSVERSIONINFOEXW vi = {sizeof(vi), major, minor, 0, 0, {0}, servpack}; - return VerifyVersionInfoW(&vi, VER_MAJORVERSION|VER_MINORVERSION|VER_SERVICEPACKMAJOR, - VerSetConditionMask(VerSetConditionMask(VerSetConditionMask(0, - VER_MAJORVERSION, VER_GREATER_EQUAL), - VER_MINORVERSION, VER_GREATER_EQUAL), - VER_SERVICEPACKMAJOR, VER_GREATER_EQUAL)); -} - -VERSIONHELPERAPI -IsWindowsXPOrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 0); -} - -VERSIONHELPERAPI -IsWindowsXPSP1OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 1); -} - -VERSIONHELPERAPI -IsWindowsXPSP2OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 2); -} - -VERSIONHELPERAPI -IsWindowsXPSP3OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINXP), LOBYTE(_WIN32_WINNT_WINXP), 3); -} - -VERSIONHELPERAPI -IsWindowsVistaOrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 0); -} - -VERSIONHELPERAPI -IsWindowsVistaSP1OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 1); -} - -VERSIONHELPERAPI -IsWindowsVistaSP2OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_VISTA), LOBYTE(_WIN32_WINNT_VISTA), 2); -} - -VERSIONHELPERAPI -IsWindows7OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN7), LOBYTE(_WIN32_WINNT_WIN7), 0); -} - -VERSIONHELPERAPI -IsWindows7SP1OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN7), LOBYTE(_WIN32_WINNT_WIN7), 1); -} - -VERSIONHELPERAPI -IsWindows8OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN8), LOBYTE(_WIN32_WINNT_WIN8), 0); -} - -VERSIONHELPERAPI -IsWindows8Point1OrGreater(void) -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINBLUE), LOBYTE(_WIN32_WINNT_WINBLUE), 0); -} - -VERSIONHELPERAPI -IsWindows10OrGreater() -{ - return IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WINTHRESHOLD), LOBYTE(_WIN32_WINNT_WINTHRESHOLD), 0); -} - -VERSIONHELPERAPI -IsWindowsServer(void) -{ - OSVERSIONINFOEXW vi = {sizeof(vi), 0, 0, 0, 0, {0}, 0, 0, 0, VER_NT_WORKSTATION}; - return !VerifyVersionInfoW(&vi, VER_PRODUCT_TYPE, VerSetConditionMask(0, VER_PRODUCT_TYPE, VER_EQUAL)); -} - -#endif /* if WINAPI_FAMILY_PARTITION(WINAPI_PARTITION_DESKTOP) && !defined(__WIDL__) */ -#endif /* ifndef _INC_VERSIONHELPERS */ diff --git a/src/openvpn/win32.c b/src/openvpn/win32.c index 47eb1fc..e998d90 100644 --- a/src/openvpn/win32.c +++ b/src/openvpn/win32.c @@ -45,11 +45,7 @@ #include "memdbg.h" -#ifdef HAVE_VERSIONHELPERS_H #include <versionhelpers.h> -#else -#include "compat-versionhelpers.h" -#endif #include "block_dns.h" diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c index a47db8a..32c8996 100644 --- a/src/openvpnserv/interactive.c +++ b/src/openvpnserv/interactive.c @@ -34,11 +34,7 @@ #include <shellapi.h> #include <mstcpip.h> -#ifdef HAVE_VERSIONHELPERS_H #include <versionhelpers.h> -#else -#include "compat-versionhelpers.h" -#endif #include "openvpn-msg.h" #include "validate.h"