aboutsummaryrefslogtreecommitdiff
path: root/build/rule.cxx
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2015-08-27 15:11:40 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2015-08-27 15:11:40 +0200
commitfd689eb883655dcb29e505b041cd02fac01f0bac (patch)
tree0d85ec32d95a1c96eaa7eff28734b900c44dd3ca /build/rule.cxx
parent7f2d06258d57e39940e8fa959336da0ea66fe37f (diff)
Dist module/meta-operation initial implementation
Diffstat (limited to 'build/rule.cxx')
-rw-r--r--build/rule.cxx14
1 files changed, 5 insertions, 9 deletions
diff --git a/build/rule.cxx b/build/rule.cxx
index 7bf030d..b9a989b 100644
--- a/build/rule.cxx
+++ b/build/rule.cxx
@@ -38,13 +38,9 @@ namespace build
// for every action (because that's the condition for us matching),
// for some actions this is clearly a waste. Say, perform_clean: we
// are not doing anything for this action so not checking if the file
- // exists seems harmless. What about, say, configure_update? Again,
- // whether we match or not, there is nothing to be done for this
- // action. And, who knows, maybe the file doesn't exist during
- // configure_update but will magically appear during perform_update.
- // So the overall guideline seems to be this: if we don't do anything
- // for the action (other than performing it on the prerequisites),
- // then we match.
+ // exists seems harmless. So the overall guideline seems to be this:
+ // if we don't do anything for the action (other than performing it
+ // on the prerequisites), then we match.
//
switch (a)
{
@@ -181,7 +177,7 @@ namespace build
{
case perform_update_id: return &perform_update;
case perform_clean_id: return &perform_clean;
- default: return default_recipe; // Forward to prerequisites.
+ default: assert (false); return default_recipe;
}
}
@@ -196,7 +192,7 @@ namespace build
if (!t.prerequisite_targets.empty ())
ts = execute_prerequisites (a, t);
- const path& d (t.dir); // Everything is in t.dir.
+ const dir_path& d (t.dir); // Everything is in t.dir.
// Generally, it is probably correct to assume that in the majority
// of cases the directory will already exist. If so, then we are