Skip to content

Commit

Permalink
CMake: Set policy to silence warnings; make minimum versions consistent
Browse files Browse the repository at this point in the history
Fixes: b/396442914
Change-Id: Icd468bb62fa9075a19a3ea685bad4e000526d068
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/267793
Lint: Lint 🤖 <[email protected]>
Commit-Queue: Auto-Submit <[email protected]>
Presubmit-Verified: CQ Bot Account <[email protected]>
Docs-Not-Needed: Wyatt Hepler <[email protected]>
Reviewed-by: Kesavan Yogeswaran <[email protected]>
Pigweed-Auto-Submit: Wyatt Hepler <[email protected]>
  • Loading branch information
255 authored and CQ Bot Account committed Feb 13, 2025
1 parent 04fb4db commit d1c3488
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# License for the specific language governing permissions and limitations under
# the License.

cmake_minimum_required(VERSION 3.16)
cmake_minimum_required(VERSION 3.20)
project(Pigweed)

# Do not rely on the PW_ROOT environment variable being set through bootstrap.
Expand Down
4 changes: 2 additions & 2 deletions pw_build/pigweed.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# the License.
include_guard(GLOBAL)

cmake_minimum_required(VERSION 3.19)
cmake_minimum_required(VERSION 3.20)

# The PW_ROOT environment variable should be set in bootstrap. If it is not set,
# set it to the root of the Pigweed repository.
Expand Down Expand Up @@ -51,7 +51,7 @@ include("$ENV{PW_ROOT}/pw_unit_test/test.cmake")
# <multi_value_keywords>)
#
macro(pw_parse_arguments)
# First parse the arguments to this macro.
cmake_policy(SET CMP0174 NEW) # Remove when CMake 3.31 or newer is required.
cmake_parse_arguments(
pw_parse_arg "" "NUM_POSITIONAL_ARGS"
"OPTION_ARGS;ONE_VALUE_ARGS;MULTI_VALUE_ARGS;REQUIRED_ARGS"
Expand Down

0 comments on commit d1c3488

Please sign in to comment.