Add a sanity check for derived tool commands

[builder.git] / source / tool.cpp

diff --git a/source/tool.cpp b/source/tool.cpp
index 852fa2d5805b294b4d74b5a5ab165039b08feaad..eaf4977d8c16030f2aa1bb0994ad3abc1d67c900 100644 (file)
--- a/source/tool.cpp
+++ b/source/tool.cpp
@@ -20,6 +20,11 @@ void Tool::set_command(const string &cmd, bool cross)
                command = cmd;
 }
 
+void Tool::set_run(function<Task *(const Target &)> f)
+{
+       run_func = move(f);
+}
+
 bool Tool::accepts_suffix(const string &suffix, bool aux) const
 {
        return (any_equals(input_suffixes, suffix) || (aux && any_equals(aux_suffixes, suffix)));
@@ -38,50 +43,34 @@ void Tool::prepare()
                return;
 
        prepared = true;
-       do_prepare();
-       if(!executable && !command.empty())
-       {
+
+       if(!command.empty())
                executable = builder.get_vfs().find_binary(command);
-               if(!executable)
-               {
-                       builder.get_logger().log("problems", "Can't find executable %s for %s", command, tag);
-                       problems.push_back(format("Can't find executable %s", command));
-               }
+       prepare(*this);
+       if(!command.empty() && !executable)
+       {
+               builder.get_logger().log("problems", "Can't find executable %s for %s", command, tag);
+               problems.push_back(format("Can't find executable %s", command));
        }
 }
 
-string Tool::create_build_signature(const BuildInfo &) const
+void Tool::prepare(Tool &tool) const
 {
-       if(executable)
-               return format("%s=%s", tag, FS::basename(executable->get_path()));
-       else
-               return string();
-}
+       if(&tool!=this && tool.get_base_tool()!=this)
+               throw invalid_argument("Tool::prepare");
 
+       if(&tool!=this && !command.empty() && tool.command.empty())
+               throw logic_error("Derived tool has no command");
 
-Target *SubTool::create_source(const Component &c, const FS::Path &p) const
-{
-       return parent.create_source(c, p);
+       do_prepare(tool);
 }
 
-Target *SubTool::create_source(const FS::Path &p) const
-{
-       return parent.create_source(p);
-}
-
-Target *SubTool::create_target(const vector<Target *> &s, const string &a)
-{
-       return parent.create_target(s, a);
-}
-
-Target *SubTool::create_install(Target &t) const
-{
-       return parent.create_install(t);
-}
-
-string SubTool::create_build_signature(const BuildInfo &bi) const
+string Tool::create_build_signature(const BuildInfo &) const
 {
-       return parent.create_build_signature(bi);
+       if(executable)
+               return format("%s=%s", tag, FS::basename(executable->get_path()));
+       else
+               return string();
 }