Skip to content

Commit 53b16d0

Browse files
flichtenheldcron2
authored andcommitted
README.cmake.md: Document minimum required CMake version for --preset
CMakePreset.json is supported since 3.19, but we have a version 3 preset file, so need at least 3.21. Github: #489 Change-Id: I44c555f6ffa08f2aee739c7f687fa3b678c86231 Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Gert Doering <[email protected]> Message-Id: <[email protected]> URL: https://www.mail-archive.com/[email protected]/msg28160.html Signed-off-by: Gert Doering <[email protected]>
1 parent bb0849d commit 53b16d0

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

README.cmake.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@ also makes cross-building with MinGW on Linux much simpler. However,
1717
builds are also possible by providing the build dependencies manually,
1818
but that might require specifying more information to CMake.
1919

20+
You need at least CMake version 3.21 or newer for the `CMakePreset.json`
21+
file to be supported. Manual builds might be possible with older CMake
22+
versions, see `cmake_minimum_required` in `CMakeLists.txt`.
23+
2024
If you're looking to build the full Windows installer MSI, take a look
2125
at https://github.com/OpenVPN/openvpn-build.git .
2226

@@ -27,7 +31,7 @@ The following tools are expected to be present on the system, you
2731
can install them with a package manager of your choice (e.g.
2832
chocolatey, winget) or manually:
2933

30-
* CMake
34+
* CMake (>= 3.21)
3135
* Git
3236
* Python (3.x), plus the Python module `docutils`
3337
* Visual Studion 17 (2022), C/C++ Enviroment
@@ -101,6 +105,7 @@ To build the Windows executables on a Linux system:
101105
git clone https://github.com/OpenVPN/openvpn.git
102106
export VCPKG_ROOT=$PWD/vcpkg
103107
cd openvpn
108+
# requires CMake 3.21 or newer
104109
cmake --preset mingw-x64
105110
cmake --build --preset mingw-x64
106111
# unit tests are built, but no testPreset is provided. You need to copy

0 commit comments

Comments
 (0)