Kitware/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2025-10-14 02:08:27 +08:00
Code Issues Projects Releases Wiki Activity

Merge topic 'ninja-omit-empty-custom-command'

Browse Source 
d43f4692e0 Ninja: Omit custom commands with an empty COMMAND

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5647
This commit is contained in:
Brad King
2021-01-05 14:01:05 +00:00
committed by Kitware Robot
parent 764ce15ffb d43f4692e0
commit 78c95e5a1c
3 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
Source/cmLocalNinjaGenerator.cxx
View File

@@ -557,9 +557,13 @@ void cmLocalNinjaGenerator::AppendCustomCommandLines(
std::string launcher = this->MakeCustomLauncher(ccg);
for (unsigned i = 0; i != ccg.GetNumberOfCommands(); ++i) {
std::string c = ccg.GetCommand(i);
if (c.empty()) {
continue;
}
cmdLines.push_back(launcher +
this->ConvertToOutputFormat(
ccg.GetCommand(i),
c,
gg->IsMultiConfig() ? cmOutputConverter::NINJAMULTI
: cmOutputConverter::SHELL));

4
Tests/CustomCommand/CMakeLists.txt
View File

@@ -582,3 +582,7 @@ set_target_properties(mac_fw PROPERTIES
)
add_custom_command(OUTPUT mac_fw.txt COMMAND ${CMAKE_COMMAND} -E touch mac_fw.txt DEPENDS mac_fw)
add_custom_target(drive_mac_fw ALL DEPENDS mac_fw.txt)
# Test empty COMMANDs are ommited
add_executable(empty_command empty_command.cxx)
add_custom_command(TARGET empty_command POST_BUILD COMMAND $<0:date>)

4
Tests/CustomCommand/empty_command.cxx Normal file
View File

@@ -0,0 +1,4 @@
int main()
{
return 0;
}