X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcomponent.cpp;h=db78917c70fdf63fe1aef7fb95fa599629ca34f9;hb=a957405689fafa1afc233182a3756e36ea34281c;hp=8a57b6410ee47024cec0b88704f9998707130a4b;hpb=73b8ae52a68a9475834745cd8514562703103677;p=builder.git

diff --git a/source/component.cpp b/source/component.cpp
index 8a57b64..db78917 100644
--- a/source/component.cpp
+++ b/source/component.cpp
@@ -101,7 +101,6 @@ void Component::create_targets() const
 	const Toolchain &toolchain = builder.get_toolchain();
 
 	SourceList source_filenames = collect_source_files();
-	list<Target *> inst_list;
 
 	string inst_loc;
 	if(type==TARBALL)
@@ -138,7 +137,8 @@ void Component::create_targets() const
 	}
 	else if(type==INSTALL)
 	{
-		inst_loc = name;
+		Target *inst = builder.get_target("install");
+		const Tool &copy = toolchain.get_tool("CP");
 		for(SourceList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
 		{
 			FileTarget *ft;
@@ -146,7 +146,7 @@ void Component::create_targets() const
 				ft = dynamic_cast<FileTarget *>(tgt);
 			else
 				ft = new File(builder, package, *i);
-			inst_list.push_back(ft);
+			inst->add_dependency(*copy.create_target(*ft, name));
 		}
 	}
 	else if(type==DATAFILE)
@@ -162,7 +162,7 @@ void Component::create_targets() const
 
 		builder.add_primary_target(*result);
 		if(install)
-			inst_list.push_back(result);
+			builder.add_installed_target(*result);
 	}
 
 	if(type==PROGRAM || type==LIBRARY || type==MODULE)
@@ -185,7 +185,7 @@ void Component::create_targets() const
 				}
 
 				if(type==LIBRARY && install && dynamic_cast<FileTarget *>(src)->is_installable())
-					inst_list.push_back(src);
+					builder.add_installed_target(*src);
 			}
 		}
 
@@ -225,17 +225,9 @@ void Component::create_targets() const
 		{
 			builder.add_primary_target(**i);
 			if(install)
-				inst_list.push_back(*i);
+				builder.add_installed_target(**i);
 		}
 	}
-
-	Target *inst_tgt = builder.get_target("install");
-	const Tool &copy = toolchain.get_tool("CP");
-	for(list<Target *>::const_iterator i=inst_list.begin(); i!=inst_list.end(); ++i)
-	{
-		Target *inst = copy.create_target(**i, inst_loc);
-		inst_tgt->add_dependency(*inst);
-	}
 }
 
 Component::SourceList Component::collect_source_files() const
@@ -302,5 +294,5 @@ void Component::Loader::use(const string &n)
 			obj.uses.push_back(&*i);
 			return;
 		}
-	error(format("Unknown library component '%s'", n));
+	throw invalid_argument("Component::Loader::use");
 }