]> git.tdb.fi Git - builder.git/blobdiff - source/lib/builder.cpp
Avoid double deletion of the module if a plugin fails to load
[builder.git] / source / lib / builder.cpp
index 92924ebc0956aee33680624e97deb5cabd6638cb..738361bce2d3553db15844bc19b7e6e5a7353924 100644 (file)
 #include <msp/strings/format.h>
 #include <msp/time/timedelta.h>
 #include <msp/time/utils.h>
-#include "android/androidtools.h"
 #include "binarypackage.h"
 #include "builder.h"
-#include "builtin/builtintools.h"
-#include "clang/clangtools.h"
-#include "datafile/datatool.h"
-#include "gnu/gnutools.h"
 #include "installedfile.h"
-#include "msvc/microsofttools.h"
 #include "package.h"
+#include "plugin.h"
 #include "sharedlibrary.h"
-#include "sourcepackage.h"
 #include "task.h"
+#include "tool.h"
 #include "virtualtarget.h"
 
 using namespace std;
@@ -46,6 +41,47 @@ Builder::~Builder()
                delete current_arch;
 }
 
+void Builder::load_plugins()
+{
+       using CreateFunc = Plugin *(Builder &);
+
+       FS::Path plugins_dir = FS::get_sys_lib_dir();
+       logger->log("files", "Traversing %s", plugins_dir);
+       for(const string &f: list_filtered(plugins_dir, "\\.dlm$"))
+       {
+               LoadedPlugin plugin;
+               plugin.path = plugins_dir/f;
+
+               try
+               {
+                       plugin.module = new Module(plugin.path.str());
+               }
+               catch(const exception &exc)
+               {
+                       logger->log("plugins", "Failed to load plugin %s: %s", f, exc.what());
+                       continue;
+               }
+
+               try
+               {
+                       CreateFunc *create_func = reinterpret_cast<CreateFunc *>(plugin.module->get_symbol("create_plugin"));
+                       plugin.plugin = create_func(*this);
+                       if(plugin.plugin)
+                       {
+                               logger->log("plugins", "Loaded plugin %s", f);
+                               plugins.emplace_back(move(plugin));
+                               continue;
+                       }
+                       else
+                               logger->log("plugins", "Plugin %s refused to initialize", f);
+               }
+               catch(const exception &exc)
+               {
+                       logger->log("plugins", "Failed to initialize plugin %s: %s", f, exc.what());
+               }
+       }
+}
+
 void Builder::set_architecture(const string &name)
 {
        if(current_arch!=&native_arch)
@@ -102,14 +138,8 @@ void Builder::update_auto_prefix()
 
 void Builder::add_default_tools()
 {
-       toolchain.add_toolchain(new GnuTools(*this, *current_arch));
-       toolchain.add_toolchain(new ClangTools(*this, *current_arch));
-       if(current_arch->get_system()=="android")
-               toolchain.add_toolchain(new AndroidTools(*this, *current_arch));
-       if(current_arch->get_system()=="windows")
-               toolchain.add_toolchain(new MicrosoftTools(*this, *current_arch));
-       toolchain.add_toolchain(new BuiltinTools(*this));
-       toolchain.add_tool(new DataTool(*this));
+       for(const LoadedPlugin &p: plugins)
+               p.plugin->add_tools(toolchain, *current_arch);
 
        auto i = find_if(toolchain.get_toolchains(), [](const Toolchain *tc){ return (tc->has_tool("CC") || tc->has_tool("CXX")); });
        if(i!=toolchain.get_toolchains().end())
@@ -127,46 +157,71 @@ void Builder::set_logger(const Logger *l)
 
 vector<string> Builder::collect_problems() const
 {
+       vector<string> target_problems;
        vector<string> problems;
-       set<const Package *> broken_packages;
-       set<const Component *> broken_components;
-       set<const Tool *> broken_tools;
+       vector<const Package *> broken_packages;
+       vector<const Component *> broken_components;
+       vector<const Tool *> broken_tools;
 
        for(const auto &kvp: build_graph.get_targets())
                if(kvp.second->is_broken())
                {
-                       for(const string &p: kvp.second->get_problems())
-                               problems.push_back(format("%s: %s", kvp.second->get_name(), p));
-
                        const Package *package = kvp.second->get_package();
-                       if(package && !package->get_problems().empty())
-                               broken_packages.insert(package);
+                       if(package && package->is_broken())
+                               collect_broken_packages(*package, broken_packages);
 
                        const Component *component = kvp.second->get_component();
-                       if(component && !component->get_problems().empty())
-                               broken_components.insert(component);
+                       if(component && component->is_broken() && !any_equals(broken_components, component))
+                       {
+                               broken_components.push_back(component);
+                               collect_broken_packages(component->get_package(), broken_packages);
+                               for(const Package *r: component->get_required_packages())
+                                       if(r->is_broken())
+                                               collect_broken_packages(*r, broken_packages);
+                       }
 
                        const Tool *tool = kvp.second->get_tool();
-                       if(tool && !tool->get_problems().empty())
-                               broken_tools.insert(tool);
+                       if(tool && tool->is_broken() && !any_equals(broken_tools, tool))
+                       {
+                               broken_tools.push_back(tool);
+                               for(const string &p: tool->get_problems())
+                                       problems.push_back(format("%s: %s", tool->get_tag(), p));
+                       }
+
+                       for(const string &p: kvp.second->get_problems())
+                               target_problems.push_back(format("%s: %s", kvp.second->get_name(), p));
                }
 
-       // TODO Sort components after their packages, and targets last
        for(const Package *p: broken_packages)
+       {
                for(const string &b: p->get_problems())
                        problems.push_back(format("%s: %s", p->get_name(), b));
 
-       for(const Component *c: broken_components)
-               for(const string &b: c->get_problems())
-                       problems.push_back(format("%s/%s: %s", c->get_package().get_name(), c->get_name(), b));
+               for(const Component *c: broken_components)
+                       if(&c->get_package()==p)
+                       {
+                               for(const string &b: c->get_problems())
+                                       problems.push_back(format("%s/%s: %s", p->get_name(), c->get_name(), b));
+                       }
+       }
 
-       for(const Tool *t: broken_tools)
-               for(const string &b: t->get_problems())
-                       problems.push_back(format("%s: %s", t->get_tag(), b));
+       problems.insert(problems.end(), make_move_iterator(target_problems.begin()), make_move_iterator(target_problems.end()));
 
        return problems;
 }
 
+void Builder::collect_broken_packages(const Package &pkg, vector<const Package *> &broken) const
+{
+       if(any_equals(broken, &pkg))
+               return;
+
+       broken.push_back(&pkg);
+
+       for(const Package *r: pkg.get_required_packages())
+               if(r->is_broken())
+                       collect_broken_packages(*r, broken);
+}
+
 void Builder::load_build_file(const FS::Path &fn, const Config::InputOptions *opts, bool all)
 {
        IO::BufferedFile in(fn.str());
@@ -310,6 +365,22 @@ int Builder::clean(bool all, bool dry_run)
 }
 
 
+Builder::LoadedPlugin::LoadedPlugin(LoadedPlugin &&other):
+       path(move(other.path)),
+       module(other.module),
+       plugin(other.plugin)
+{
+       other.module = 0;
+       other.plugin = 0;
+}
+
+Builder::LoadedPlugin::~LoadedPlugin()
+{
+       delete plugin;
+       delete module;
+}
+
+
 Builder::Loader::Loader(Builder &b, const Config::InputOptions *o, bool a):
        DataFile::ObjectLoader<Builder>(b),
        options(o),