From e82e2c38c1ce4fef2962c6b5ee1de0ca90f3aa60 Mon Sep 17 00:00:00 2001 From: Craig Scott Date: Fri, 7 Jun 2024 17:16:13 +1000 Subject: [PATCH] Tests: RunCMake.FetchContent should not always force _deps removal To properly test some functionality, tests may rely on not clearing things like time stamps between cmake invocations. The RunCMake infrastructure clears the build directory by default anyway, and tests may individually ask for that to be disabled where needed. The line being removed here was originally added to assist with manually re-running individual tests locally outside the control of RunCMake. That is no longer appropriate. --- Tests/RunCMake/FetchContent/CMakeLists.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/Tests/RunCMake/FetchContent/CMakeLists.txt b/Tests/RunCMake/FetchContent/CMakeLists.txt index 04d024b012..4f2de6c924 100644 --- a/Tests/RunCMake/FetchContent/CMakeLists.txt +++ b/Tests/RunCMake/FetchContent/CMakeLists.txt @@ -1,9 +1,6 @@ cmake_minimum_required(VERSION 3.9) project(${RunCMake_TEST} NONE) -# Tests assume no previous downloads in the output directory -file(REMOVE_RECURSE ${CMAKE_CURRENT_BINARY_DIR}/_deps) - if(CMP0168 STREQUAL "NEW") cmake_policy(SET CMP0168 NEW) else()