Make is original and don't change anything to make it affect please On Mon, Nov 20, 2023, 2:52 AM plaisthos (Code Review) <ger...@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-open...@rfc2549.org> > Gerrit-Reviewer: flichtenheld <fr...@lichtenheld.com> > Gerrit-CC: openvpn-devel <openvpn-devel@lists.sourceforge.net> > Gerrit-Attention: flichtenheld <fr...@lichtenheld.com> > Gerrit-MessageType: newchange > _______________________________________________ > Openvpn-devel mailing list > Openvpn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/openvpn-devel >
_______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel