From c1170b5602d609f6ccd2506e5db9916c74388187 Mon Sep 17 00:00:00 2001 From: Robert Maynard Date: Wed, 11 Jan 2023 16:19:09 -0500 Subject: cmake: Add -E copy_directory_if_different Fixes #21584 --- Help/manual/cmake.1.rst | 10 ++++++++++ Help/release/dev/cmake-E-copy-directory-if-different.rst | 4 ++++ Source/cmcmd.cxx | 11 ++++++++--- Tests/RunCMake/CommandLine/RunCMakeTest.cmake | 9 +++++++++ 4 files changed, 31 insertions(+), 3 deletions(-) create mode 100644 Help/release/dev/cmake-E-copy-directory-if-different.rst diff --git a/Help/manual/cmake.1.rst b/Help/manual/cmake.1.rst index dc51383..9f77562 100644 --- a/Help/manual/cmake.1.rst +++ b/Help/manual/cmake.1.rst @@ -876,6 +876,16 @@ Available commands are: The command now fails when the source directory does not exist. Previously it succeeded by creating an empty destination directory. +.. option:: copy_directory_if_different ... + + .. versionadded:: 3.26 + + Copy changed content of ``...`` directories to ```` directory. + If ```` directory does not exist it will be created. + + ``copy_directory_if_different`` does follow symlinks. + The command fails when the source directory does not exist. + .. option:: copy_if_different ... Copy files to ```` (either file or directory) if diff --git a/Help/release/dev/cmake-E-copy-directory-if-different.rst b/Help/release/dev/cmake-E-copy-directory-if-different.rst new file mode 100644 index 0000000..6e642c0 --- /dev/null +++ b/Help/release/dev/cmake-E-copy-directory-if-different.rst @@ -0,0 +1,4 @@ +cmake-E-copy-directory-if-different +----------------------------------- + +* The :manual:`cmake(1)` ``-E`` option learned a new ``copy_directory_if_different`` command. diff --git a/Source/cmcmd.cxx b/Source/cmcmd.cxx index 4303f96..21d0cc9 100644 --- a/Source/cmcmd.cxx +++ b/Source/cmcmd.cxx @@ -110,6 +110,8 @@ void CMakeCommandUsage(std::string const& program) "(either file or directory)\n" << " copy_directory ... destination - copy content of ... " "directories to 'destination' directory\n" + << " copy_directory_if_different ... destination - copy changed content of ... " + "directories to 'destination' directory\n" << " copy_if_different ... destination - copy files if it has " "changed\n" << " echo [...] - displays arguments as text\n" @@ -731,12 +733,15 @@ int cmcmd::ExecuteCMakeCommand(std::vector const& args, return return_value; } - // Copy directory content - if (args[1] == "copy_directory" && args.size() > 3) { + // Copy directory contents + if ((args[1] == "copy_directory" || + args[1] == "copy_directory_if_different") && + args.size() > 3) { // If error occurs we want to continue copying next files. bool return_value = false; + const bool copy_always = (args[1] == "copy_directory"); for (auto const& arg : cmMakeRange(args).advance(2).retreat(1)) { - if (!cmSystemTools::CopyADirectory(arg, args.back())) { + if (!cmSystemTools::CopyADirectory(arg, args.back(), copy_always)) { std::cerr << "Error copying directory from \"" << arg << "\" to \"" << args.back() << "\".\n"; return_value = true; diff --git a/Tests/RunCMake/CommandLine/RunCMakeTest.cmake b/Tests/RunCMake/CommandLine/RunCMakeTest.cmake index 480ad09..f48e845 100644 --- a/Tests/RunCMake/CommandLine/RunCMakeTest.cmake +++ b/Tests/RunCMake/CommandLine/RunCMakeTest.cmake @@ -594,6 +594,15 @@ unset(in) unset(out) set(in ${RunCMake_SOURCE_DIR}/copy_input) +set(out ${RunCMake_BINARY_DIR}/copy_directory_different_output) +file(REMOVE_RECURSE "${out}") +file(MAKE_DIRECTORY ${out}) +run_cmake_command(E_copy_directory_if_different + ${CMAKE_COMMAND} -E copy_directory_if_different ${in} ${out}) +unset(in) +unset(out) + +set(in ${RunCMake_SOURCE_DIR}/copy_input) set(out ${RunCMake_BINARY_DIR}/copy_directory_output) set(outfile ${out}/file_for_test.txt) file(REMOVE_RECURSE "${out}") -- cgit v0.12