X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fdatatool.cpp;h=ca2f2e49aaef3738cd2bf2c12171af03d902c746;hb=7c2db9e2b91da953701be233336c5bfa1f3c4af0;hp=a956984a35deb6e9aa8b19eeca274fab4513db37;hpb=d334fc1d04b7f83c7a2e9f16439fec884f4de471;p=builder.git

diff --git a/source/datatool.cpp b/source/datatool.cpp
index a956984..ca2f2e4 100644
--- a/source/datatool.cpp
+++ b/source/datatool.cpp
@@ -1,6 +1,5 @@
 #include <stdexcept>
 #include <msp/fs/utils.h>
-#include <msp/strings/format.h>
 #include "builder.h"
 #include "component.h"
 #include "datacollection.h"
@@ -16,10 +15,7 @@ using namespace Msp;
 DataTool::DataTool(Builder &b):
 	Tool(b, "DATA")
 {
-	executable = builder.get_vfs().find_binary("mspdatatool");
-	if(!executable)
-		builder.problem(string(), format("Can't find executable mspdatatool for tool %s", tag));
-
+	set_command("mspdatatool");
 	input_suffixes.push_back(".mdt");
 }
 
@@ -28,7 +24,7 @@ Target *DataTool::create_source(const Component &comp, const FS::Path &path) con
 	return new DataTransform(builder, comp, path);
 }
 
-Target *DataTool::create_target(const list<Target *> &sources, const string &arg) const
+Target *DataTool::create_target(const list<Target *> &sources, const string &arg)
 {
 	if(arg=="collection")
 	{
@@ -42,8 +38,8 @@ Target *DataTool::create_target(const list<Target *> &sources, const string &arg
 	else if(arg=="pack")
 	{
 		list<FileTarget *> files;
-		for(list<Target *>::const_iterator i=sources.begin(); i!=sources.end(); ++i)
-			files.push_back(&dynamic_cast<FileTarget &>(**i));
+		for(Target *t: sources)
+			files.push_back(&dynamic_cast<FileTarget &>(*t));
 		DataPack *pack = new DataPack(builder, *files.front()->get_component(), files);
 		pack->set_tool(*this);
 		return pack;
@@ -52,6 +48,20 @@ Target *DataTool::create_target(const list<Target *> &sources, const string &arg
 		throw invalid_argument("DataTool::create_target");
 }
 
+string DataTool::create_build_signature(const BuildInfo &binfo) const
+{
+	string result;
+	if(binfo.debug)
+		result += 'g';
+	if(binfo.optimize>0)
+	{
+		result += 'b';
+		if(binfo.optimize>1)
+			result += 'z';
+	}
+	return result;
+}
+
 Task *DataTool::run(const Target &tgt) const
 {
 	const Component &comp = *tgt.get_component();
@@ -63,7 +73,8 @@ Task *DataTool::run(const Target &tgt) const
 	argv.push_back("-o");
 	argv.push_back(FS::relative(dynamic_cast<const FileTarget &>(tgt).get_path(), work_dir).str());
 
-	const BuildInfo &binfo = comp.get_build_info();
+	BuildInfo binfo;
+	tgt.collect_build_info(binfo);
 	if(binfo.debug)
 		argv.push_back("-g");
 	if(binfo.optimize>0)
@@ -81,9 +92,8 @@ Task *DataTool::run(const Target &tgt) const
 	else if(const DataPack *pack = dynamic_cast<const DataPack *>(&tgt))
 	{
 		argv.push_back("-p");
-		const DataPack::FileList &files = pack->get_files();
-		for(DataPack::FileList::const_iterator i=files.begin(); i!=files.end(); ++i)
-			argv.push_back(FS::relative((*i)->get_path(), work_dir).str());
+		for(const FileTarget *f: pack->get_files())
+			argv.push_back(FS::relative(f->get_path(), work_dir).str());
 	}
 
 	return new ExternalTask(argv, work_dir);