X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcomponent.cpp;h=0186f3e394c3debe464a1d888ad12594fb941f0b;hb=d1f9551e05c9d341149eb490e05b1465d3d6b711;hp=963625122005f7fdff8837ba5372deaf0cd4f02a;hpb=b45cfe5e437ca79bb3176618769628c58c0734d1;p=builder.git

diff --git a/source/component.cpp b/source/component.cpp
index 9636251..0186f3e 100644
--- a/source/component.cpp
+++ b/source/component.cpp
@@ -1,3 +1,4 @@
+#include <deque>
 #include <msp/core/algorithm.h>
 #include <msp/fs/dir.h>
 #include <msp/fs/stat.h>
@@ -12,9 +13,7 @@ using namespace Msp;
 
 Component::Component(SourcePackage &p, const string &n):
 	package(p),
-	name(n),
-	install(false),
-	deflt(true)
+	name(n)
 { }
 
 void Component::prepare()
@@ -30,18 +29,23 @@ void Component::create_build_info()
 	const Package::Requirements &pkg_reqs = package.get_required_packages();
 	Package::Requirements direct_reqs = requires;
 	direct_reqs.insert(direct_reqs.end(), pkg_reqs.begin(), pkg_reqs.end());
+	for(Package *r: direct_reqs)
+		final_build_info.update_from(r->get_exported_build_info(), BuildInfo::DEPENDENCY);
 
 	Package::Requirements all_reqs = direct_reqs;
-	for(Package *r: all_reqs)
+	deque<Package *> queue(direct_reqs.begin(), direct_reqs.end());
+	while(!queue.empty())
 	{
-		BuildInfo::UpdateLevel level = BuildInfo::CHAINED;
-		if(any_equals(direct_reqs, r))
-			level = BuildInfo::DEPENDENCY;
-		final_build_info.update_from(r->get_exported_build_info(), level);
-
-		for(Package *q: r->get_required_packages())
-			if(!any_equals(all_reqs, q))
-				all_reqs.push_back(q);
+		Package *req = queue.front();
+		queue.pop_front();
+
+		for(Package *r: req->get_required_packages())
+			if(!any_equals(all_reqs, r))
+			{
+				final_build_info.update_from(r->get_exported_build_info(), BuildInfo::CHAINED);
+				all_reqs.push_back(r);
+				queue.push_back(r);
+			}
 	}
 
 	final_build_info.update_from(package.get_build_info());
@@ -88,14 +92,15 @@ BuildInfo Component::get_build_info_for_path(const FS::Path &path) const
 	return binfo;
 }
 
-Component::SourceList Component::collect_source_files() const
+vector<FS::Path> Component::collect_source_files() const
 {
-	SourceList files;
+	vector<FS::Path> files;
 	for(const FS::Path &p: sources)
 	{
 		if(FS::is_dir(p))
 		{
-			SourceList dirs;
+			vector<FS::Path> dirs;
+			dirs.reserve(1+overlays.size());
 			dirs.push_back(p);
 			for(const string &o: overlays)
 			{
@@ -106,7 +111,7 @@ Component::SourceList Component::collect_source_files() const
 			set<string> overlay_files;
 			for(auto j=dirs.begin(); j!=dirs.end(); ++j)
 			{
-				package.get_builder().get_logger().log("files", format("Traversing %s", *j));
+				package.get_builder().get_logger().log("files", "Traversing %s", *j);
 				for(const string &f: list_files(*j))
 				{
 					if(j!=dirs.begin())