X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fmingwdlltool.cpp;h=62ef7059a885d7a78d2dd216b2d2612c3eeabd42;hb=1ed833343bc83b83c5f61cbfd74423bbba677a04;hp=5f62a02c954dadbcbc808b15b37d55938d90d812;hpb=a957405689fafa1afc233182a3756e36ea34281c;p=builder.git

diff --git a/source/mingwdlltool.cpp b/source/mingwdlltool.cpp
index 5f62a02..62ef705 100644
--- a/source/mingwdlltool.cpp
+++ b/source/mingwdlltool.cpp
@@ -1,7 +1,6 @@
 #include <cstdlib>
 #include <msp/fs/utils.h>
 #include <msp/strings/format.h>
-#include "architecture.h"
 #include "builder.h"
 #include "component.h"
 #include "exportdefinitions.h"
@@ -19,22 +18,20 @@ using namespace Msp;
 MingwDllTool::MingwDllTool(Builder &b, const Architecture &a):
 	Tool(b, a, "DLL")
 {
-	string command = "dlltool";
-	if(architecture->is_cross())
-		command = format("%s-%s", architecture->get_cross_prefix(), command);
-	executable = builder.get_vfs().find_binary(command);
+	set_command("dlltool", true);
+	set_run(_run);
 }
 
-Target *MingwDllTool::create_target(const list<Target *> &sources, const string &) const
+Target *MingwDllTool::create_target(const vector<Target *> &sources, const string &)
 {
 	if(sources.size()!=1)
 		throw invalid_argument("MingwDllTool::create_target");
 	SharedLibrary &shlib = dynamic_cast<SharedLibrary &>(*sources.front());
 
-	list<ObjectFile *> objs;
-	const Target::Dependencies &depends = shlib.get_dependencies();
-	for(Target::Dependencies::const_iterator i=depends.begin(); i!=depends.end(); ++i)
-		if(ObjectFile *obj = dynamic_cast<ObjectFile *>(*i))
+	vector<ObjectFile *> objs;
+	objs.reserve(shlib.get_dependencies().size());
+	for(Target *d: shlib.get_dependencies())
+		if(ObjectFile *obj = dynamic_cast<ObjectFile *>(d))
 			objs.push_back(obj);
 
 	ExportDefinitions *exp = new ExportDefinitions(builder, *shlib.get_component(), objs);
@@ -50,7 +47,7 @@ Target *MingwDllTool::create_install(Target &target) const
 {
 	if(ImportLibrary *imp = dynamic_cast<ImportLibrary *>(&target))
 	{
-		const Tool &copy = builder.get_toolchain().get_tool("CP");
+		Tool &copy = builder.get_toolchain().get_tool("CP");
 		InstalledFile *inst_tgt = dynamic_cast<InstalledFile *>(copy.create_target(target));
 		string link_name = format("lib%s.dll.a", imp->get_shared_library()->get_libname());
 		if(link_name!=FS::basename(inst_tgt->get_path()))
@@ -61,8 +58,10 @@ Target *MingwDllTool::create_install(Target &target) const
 		return 0;
 }
 
-Task *MingwDllTool::run(const Target &target) const
+Task *MingwDllTool::_run(const Target &target)
 {
+	const Tool &tool = *target.get_tool();
+
 	const ImportLibrary *imp = dynamic_cast<const ImportLibrary *>(&target);
 	const ExportDefinitions *exp = 0;
 	if(imp)
@@ -73,7 +72,7 @@ Task *MingwDllTool::run(const Target &target) const
 		throw invalid_argument("MingwDllTool::run");
 
 	vector<string> argv;
-	argv.push_back(executable->get_path().str());
+	argv.push_back(tool.get_executable()->get_path().str());
 
 	/* dlltool is stupid and puts temporary files in the working directory by
 	default */
@@ -104,12 +103,9 @@ Task *MingwDllTool::run(const Target &target) const
 	}
 	else
 	{
-		const Target::Dependencies &depends = exp->get_dependencies();
-		for(Target::Dependencies::const_iterator i=depends.begin(); i!=depends.end(); ++i)
-		{
-			if(ObjectFile *obj = dynamic_cast<ObjectFile *>(*i))
+		for(Target *d: exp->get_dependencies())
+			if(ObjectFile *obj = dynamic_cast<ObjectFile *>(d))
 				argv.push_back(relative(obj->get_path(), work_dir).str());
-		}
 
 		// XXX Should use dllexport, but that has some other problems to solve
 		argv.push_back("--export-all-symbols");