Skip to content

Commit

Permalink
Remove cmake_minimum_required from toolchain files
Browse files Browse the repository at this point in the history
This command is generally not included in toolchain files. It is used in
project configuration files that will consume the toolchain file. Also,
such a low value triggers warnings, support for <3.5.0 is being removed
from CMake.
  • Loading branch information
thewtex committed Apr 15, 2024
1 parent b4f426a commit aee9c61
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
3 changes: 0 additions & 3 deletions wasi-sdk-pthread.cmake
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
# Cmake toolchain description file for the Makefile

# This is arbitrary, AFAIK, for now.
cmake_minimum_required(VERSION 3.4.0)

set(CMAKE_SYSTEM_NAME WASI)
set(CMAKE_SYSTEM_VERSION 1)
set(CMAKE_SYSTEM_PROCESSOR wasm32)
Expand Down
3 changes: 0 additions & 3 deletions wasi-sdk.cmake
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
# Cmake toolchain description file for the Makefile

# This is arbitrary, AFAIK, for now.
cmake_minimum_required(VERSION 3.4.0)

# Until Platform/WASI.cmake is upstream we need to inject the path to it
# into CMAKE_MODULE_PATH.
list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}")
Expand Down

0 comments on commit aee9c61

Please sign in to comment.