X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fdatatool.cpp;h=d23c00de00f8531898c7278d50db78730e88ddc0;hb=e9e9b639fbbf500866ec1814dee6b504bcb99f3d;hp=9b101c7bf6187332636615af74a058ed06127c2f;hpb=e516e78510ad455eebc0e06645c9f12943117525;p=builder.git

diff --git a/source/datatool.cpp b/source/datatool.cpp
index 9b101c7..d23c00d 100644
--- a/source/datatool.cpp
+++ b/source/datatool.cpp
@@ -37,9 +37,11 @@ Target *DataTool::create_target(const list<Target *> &sources, const string &arg
 	}
 	else if(arg=="pack")
 	{
+		if(sources.empty())
+			throw invalid_argument("DataTool::create_target");
 		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;
@@ -48,6 +50,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();
@@ -59,7 +75,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)
@@ -77,9 +94,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);