From 722953d14c16596bf4316d3c6eda18b3a17eb693 Mon Sep 17 00:00:00 2001 From: Peter Collingbourne Date: Fri, 25 Nov 2011 15:18:49 +0000 Subject: Have the clean tool remove depfiles --- src/clean.cc | 2 ++ src/clean_test.cc | 15 +++++++++++++++ 2 files changed, 17 insertions(+) diff --git a/src/clean.cc b/src/clean.cc index 033fa96..a89e1f5 100644 --- a/src/clean.cc +++ b/src/clean.cc @@ -111,6 +111,8 @@ int Cleaner::CleanAll(bool generator) { out_node != (*e)->outputs_.end(); ++out_node) { Remove((*out_node)->file_->path_); } + if (!(*e)->rule_->depfile_.empty()) + Remove((*e)->EvaluateDepFile()); } PrintFooter(); return status_; diff --git a/src/clean_test.cc b/src/clean_test.cc index 8606239..9cd4a95 100644 --- a/src/clean_test.cc +++ b/src/clean_test.cc @@ -234,6 +234,21 @@ TEST_F(CleanTest, CleanRuleGenerator) { EXPECT_EQ(2u, fs_.files_removed_.size()); } +TEST_F(CleanTest, CleanDepFile) { + ASSERT_NO_FATAL_FAILURE(AssertParse(&state_, +"rule cc\n" +" command = cc $in > $out\n" +" depfile = $out.d\n" +"build out1: cc in1\n")); + fs_.Create("out1", 1, ""); + fs_.Create("out1.d", 1, ""); + + Cleaner cleaner(&state_, config_, &fs_); + EXPECT_EQ(0, cleaner.CleanAll()); + EXPECT_EQ(2, cleaner.cleaned_files_count()); + EXPECT_EQ(2u, fs_.files_removed_.size()); +} + TEST_F(CleanTest, CleanFailure) { ASSERT_NO_FATAL_FAILURE(AssertParse(&state_, "build dir: cat src1\n")); -- cgit v0.12